Problem/Motivation
phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig modules/contrib/font_resize/
FILE: /var/www/html/VB/drupal-10.1.0/modules/contrib/font_resize/src/Plugin/Block/ResizeBlock.php
-------------------------------------------------------------------------------------------------------------------------------------------
FOUND 4 ERRORS AND 3 WARNINGS AFFECTING 6 LINES
-------------------------------------------------------------------------------------------------------------------------------------------
22 | WARNING | [ ] Unused variable $label_plus.
23 | WARNING | [ ] Unused variable $label_minus.
24 | WARNING | [ ] Unused variable $label_original.
32 | ERROR | [x] Short array syntax must be used to define arrays
34 | ERROR | [x] Short array syntax must be used to define arrays
35 | ERROR | [x] Short array syntax must be used to define arrays
35 | ERROR | [ ] The array declaration extends to column 88 (the limit is 80). The array content should be split up over multiple lines
-------------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 3 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-------------------------------------------------------------------------------------------------------------------------------------------
FILE: /var/www/html/VB/drupal-10.1.0/modules/contrib/font_resize/README.md
--------------------------------------------------------------------------
FOUND 1 ERROR AND 1 WARNING AFFECTING 2 LINES
--------------------------------------------------------------------------
3 | WARNING | [ ] Line exceeds 80 characters; contains 82 characters
21 | ERROR | [x] Expected 1 newline at end of file; 0 found
Steps to reproduce
phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig modules/contrib/font_resize/
Proposed resolution
Remaining tasks
User interface changes
API changes
Data model changes
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | after.png | 70.17 KB | paraderojether |
| #4 | before.png | 203.9 KB | paraderojether |
Issue fork font_resize-3372129
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
- phpcs
changes, plain diff MR !1
Comments
Comment #3
indrapatil commentedComment #4
paraderojether commentedHi
I reviewed MR!1, and confirmed it fixed the issues reported by phpcs.
I added screenshots for reference.
Thank You.
Comment #5
joseph.olstadMerge conflict.
Closing as works as designed.
Comment #6
joseph.olstad