Conversation
| end | ||
|
|
||
| end | ||
|
|
There was a problem hiding this comment.
Extra empty line detected at module body end.
| encoder | ||
|
|
||
| end | ||
|
|
There was a problem hiding this comment.
Extra empty line detected at class body end.
| end | ||
|
|
||
| encoder | ||
|
|
There was a problem hiding this comment.
Extra empty line detected at method body end.
|
|
||
| end | ||
|
|
||
| if match = scan(/\n/) |
There was a problem hiding this comment.
Assignment in condition - you probably meant to use ==.
| end | ||
|
|
||
| elsif match = scan(/.+/) | ||
| @html_scanner.tokenize match, :tokens => encoder |
| @html_scanner.tokenize self[2], :tokens => encoder if self[2] | ||
| end | ||
|
|
||
| elsif match = scan(/.+/) |
There was a problem hiding this comment.
Assignment in condition - you probably meant to use ==.
| elsif match = scan(/((?:<|><?)(?![!?\/\w]))?(.+)?/) | ||
| encoder.text_token self[1], :plain if self[1] | ||
| # TODO: recognize #{...} snippets | ||
| @html_scanner.tokenize self[2], :tokens => encoder if self[2] |
There was a problem hiding this comment.
Avoid more than 3 levels of block nesting.
Use the new Ruby 1.9 hash syntax.
| end | ||
| end | ||
| elsif match = scan(/((?:<|><?)(?![!?\/\w]))?(.+)?/) | ||
| encoder.text_token self[1], :plain if self[1] |
There was a problem hiding this comment.
Avoid more than 3 levels of block nesting.
| @ruby_scanner.tokenize self[4], :tokens => encoder | ||
| end | ||
| end | ||
| elsif match = scan(/((?:<|><?)(?![!?\/\w]))?(.+)?/) |
There was a problem hiding this comment.
Assignment in condition - you probably meant to use ==.
Use %r around regular expression.
| if self[2] | ||
| @embedded_ruby_scanner.tokenize self[4], :tokens => encoder | ||
| else | ||
| @ruby_scanner.tokenize self[4], :tokens => encoder |
| encoder.text_token self[1] + self[3], :plain | ||
| if self[4] | ||
| if self[2] | ||
| @embedded_ruby_scanner.tokenize self[4], :tokens => encoder |
|
|
||
| if scan(/(>?<?[-=]|[&!]=|(& |!)|~)( *)([^,\n\|]+(?:(, *|\|(?=.|\n.*\|$))\n?[^,\n\|]*)*)?/) | ||
| encoder.text_token self[1] + self[3], :plain | ||
| if self[4] |
There was a problem hiding this comment.
Avoid more than 3 levels of block nesting.
| encoder.text_token match, :tag | ||
| end | ||
|
|
||
| if scan(/(>?<?[-=]|[&!]=|(& |!)|~)( *)([^,\n\|]+(?:(, *|\|(?=.|\n.*\|$))\n?[^,\n\|]*)*)?/) |
| encoder.text_token self[3], :plain if self[3] | ||
| end | ||
|
|
||
| if tag && match = scan(/\//) |
There was a problem hiding this comment.
Assignment in condition - you probably meant to use ==.
Use %r around regular expression.
| if tag && match = scan(/(\[)([^\]\n]+)?(\])?/) | ||
| encoder.text_token self[1], :plain | ||
| @ruby_scanner.tokenize self[2], :tokens => encoder if self[2] | ||
| encoder.text_token self[3], :plain if self[3] |
There was a problem hiding this comment.
Avoid more than 3 levels of block nesting.
|
|
||
| if tag && match = scan(/(\[)([^\]\n]+)?(\])?/) | ||
| encoder.text_token self[1], :plain | ||
| @ruby_scanner.tokenize self[2], :tokens => encoder if self[2] |
There was a problem hiding this comment.
Avoid more than 3 levels of block nesting.
Use the new Ruby 1.9 hash syntax.
| encoder.text_token match, :plain if match | ||
| end | ||
|
|
||
| if tag && match = scan(/(\[)([^\]\n]+)?(\])?/) |
There was a problem hiding this comment.
Assignment in condition - you probably meant to use ==.
| end | ||
| @ruby_scanner.tokenize code, :tokens => encoder unless code.empty? | ||
|
|
||
| encoder.text_token match, :plain if match |
There was a problem hiding this comment.
Avoid more than 3 levels of block nesting.
| break | ||
| end | ||
| end | ||
| @ruby_scanner.tokenize code, :tokens => encoder unless code.empty? |
There was a problem hiding this comment.
Avoid more than 3 levels of block nesting.
Use the new Ruby 1.9 hash syntax.
| else | ||
| break | ||
| end | ||
| when "\n", ",", nil |
There was a problem hiding this comment.
Prefer single-quoted strings when you don't need string interpolation or special symbols.
jchrtek
force-pushed
the
Add-docs-for-admin
branch
from
27517c5 to
8a3b851
Compare
| class ApplicationController < ActionController::Base | ||
| layout 'playground' | ||
|
|
||
|
|
There was a problem hiding this comment.
Extra blank line detected.
Extra empty line detected at class body end.
| require_relative '../../../lib/blueberry_admin/coderay_slim.rb' | ||
|
|
||
| module BlueberryAdmin | ||
| class ApplicationController < ActionController::Base |
There was a problem hiding this comment.
Missing top-level class documentation comment.
| @@ -0,0 +1,9 @@ | |||
| require_relative '../../../lib/blueberry_admin/coderay_slim.rb' | |||
There was a problem hiding this comment.
Missing frozen string literal comment.
jchrtek
force-pushed
the
Add-docs-for-admin
branch
from
8a3b851 to
e065120
Compare
2 participants
No description provided.