-
Notifications
You must be signed in to change notification settings - Fork 2
Fixes from git.lc project #3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
Gormartsen
wants to merge
1
commit into
master
Choose a base branch
from
codereview-7164-drupal-render_boost.module
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
228:1 Line indented incorrectly; expected 4 spaces, found 0 228:1 4 spaces found before inline comment; expected "// drupal_process_states($elements);" but found "// drupal_process_states($elements);" 232:1 Line indented incorrectly; expected 4 spaces, found 0 232:1 4 spaces found before inline comment; expected "// drupal_process_attached($elements);" but found "// drupal_process_attached($elements);" 89:1 Missing function doc comment 89:46 Expected 1 space after closing parenthesis; found 0 91:1 Whitespace found at end of line 92:3 Expected 1 space after IF keyword; 0 found 92:12 Expected 1 space after closing parenthesis; found "" 94:4 Expected 1 space after ELSE keyword; 0 found 94:3 Expected newline after closing brace 95:11 There must not be a single space before a unary operator statement 97:1 Whitespace found at end of line 98:3 Expected 1 space after IF keyword; 0 found 98:15 Expected 1 space after closing parenthesis; found "" 101:1 Whitespace found at end of line 102:3 Expected 1 space after IF keyword; 0 found 102:24 Expected 1 space after closing parenthesis; found "" 118:5 Expected 1 space after FOREACH keyword; 0 found 118:39 Expected 1 space after closing parenthesis; found "" 125:1 Missing function doc comment 125:52 Expected 1 space after closing parenthesis; found 0 126:3 Expected 1 space after IF keyword; 0 found 126:22 Expected 1 space after closing parenthesis; found "" 128:5 Expected 1 space after IF keyword; 0 found 128:31 Expected 1 space after closing parenthesis; found "" 161:42 Space found before semicolon; expected "$expire_after;" but found "$expire_after ;" 166:1 Missing function doc comment 166:55 Expected 1 space after closing parenthesis; found 0 168:1 Whitespace found at end of line 169:3 Expected 1 space after IF keyword; 0 found 169:17 Expected 1 space after closing parenthesis; found "" 172:1 Whitespace found at end of line 173:3 Expected 1 space after IF keyword; 0 found 173:15 Expected 1 space after closing parenthesis; found "" 198:5 Expected 1 space after IF keyword; 0 found 198:33 Expected 1 space after closing parenthesis; found "" 199:7 Expected 1 space after FOREACH keyword; 0 found 199:49 Expected 1 space after closing parenthesis; found "" 203:5 Expected 1 space after IF keyword; 0 found 203:34 Expected 1 space after closing parenthesis; found "" 204:7 Expected 1 space after FOREACH keyword; 0 found 204:50 Expected 1 space after closing parenthesis; found "" 208:5 Expected 1 space after IF keyword; 0 found 208:39 Expected 1 space after closing parenthesis; found "" 209:7 Expected 1 space after FOREACH keyword; 0 found 209:55 Expected 1 space after closing parenthesis; found "" 213:1 Whitespace found at end of line 214:41 Concat operator must be surrounded by a single space 214:104 Concat operator must be surrounded by a single space 220:18 Space found before semicolon; expected "$return;" but found "$return ;" 245:1 Whitespace found at end of line 245:1 Functions must not contain multiple empty lines in a row; found 2 empty lines 246:1 Whitespace found at end of line 249:1 Whitespace found at end of line 252:1 Whitespace found at end of line 258:1 Whitespace found at end of line 274:1 Whitespace found at end of line 279:1 Whitespace found at end of line 309:1 Whitespace found at end of line 321:9 Expected 1 space after IF keyword; 0 found 321:72 Expected 1 space after closing parenthesis; found "" 321:50 Expected one space after the comma, 0 found 322:11 Expected 1 space after FOREACH keyword; 0 found 322:32 Expected 1 space after closing parenthesis; found "" 326:1 Whitespace found at end of line 330:9 Expected 1 space after IF keyword; 0 found 330:66 Expected 1 space after closing parenthesis; found "" 330:47 Expected one space after the comma, 0 found 331:11 Expected 1 space after FOREACH keyword; 0 found 331:32 Expected 1 space after closing parenthesis; found "" 335:1 Whitespace found at end of line 338:1 Whitespace found at end of line 339:9 Expected 1 space after IF keyword; 0 found 339:64 Expected 1 space after closing parenthesis; found "" 339:46 Expected one space after the comma, 0 found 340:11 Expected 1 space after FOREACH keyword; 0 found 340:32 Expected 1 space after closing parenthesis; found "" 345:1 Whitespace found at end of line 402:30 If the line declaring an array spans longer than 80 characters, each element should be broken into its own line 532:1 Missing function doc comment 532:72 Expected 1 space after closing parenthesis; found 0 534:3 Expected 1 space after IF keyword; 0 found 534:18 Expected 1 space after closing parenthesis; found "" 539:1 Whitespace found at end of line 541:7 Expected 1 space after FOREACH keyword; 0 found 541:43 Expected 1 space after closing parenthesis; found "" 544:7 Expected 1 space after FOREACH keyword; 0 found 544:51 Expected 1 space after closing parenthesis; found "" 546:1 Whitespace found at end of line 547:9 Expected 1 space after IF keyword; 0 found 547:40 Expected 1 space after closing parenthesis; found "" 551:9 Expected 1 space after IF keyword; 0 found 551:40 Expected 1 space after closing parenthesis; found "" 552:11 Expected 1 space after FOREACH keyword; 0 found 552:56 Expected 1 space after closing parenthesis; found "" 556:9 Expected 1 space after IF keyword; 0 found 556:41 Expected 1 space after closing parenthesis; found "" 557:11 Expected 1 space after FOREACH keyword; 0 found 557:57 Expected 1 space after closing parenthesis; found "" 561:9 Expected 1 space after IF keyword; 0 found 561:46 Expected 1 space after closing parenthesis; found "" 562:11 Expected 1 space after FOREACH keyword; 0 found 562:62 Expected 1 space after closing parenthesis; found "" 567:1 Whitespace found at end of line 568:7 Expected 1 space after FOREACH keyword; 0 found 568:39 Expected 1 space after closing parenthesis; found "" 573:62 A comma should follow the last multiline array item. Found: ) 575:6 Expected 1 space after ELSE keyword; 0 found 575:5 Expected newline after closing brace 576:1 Whitespace found at end of line 580:62 A comma should follow the last multiline array item. Found: ) 583:1 Whitespace found at end of line 584:7 Expected 1 space after IF keyword; 0 found 584:44 Expected 1 space after closing parenthesis; found "" 584:35 Expected one space after the comma, 0 found 587:48 Space found before semicolon; expected "$expire_after;" but found "$expire_after ;" 591:1 Whitespace found at end of line 593:4 Expected 1 space after ELSE keyword; 0 found 593:3 Expected newline after closing brace
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Code standard Fixes. Please check before merge.