Fix phpcs issues

/web/modules/custom/ckeditor_infocard/ FILE: /web/modules/custom/ckeditor_infocard/src/Plugin/CKEditorPlugin/CKEditorInfoCardButton.php ----------------------------------------------------------------------------------------------------------------------------------------------------------- FOUND 2 ERRORS AFFECTING 2 LINES ----------------------------------------------------------------------------------------------------------------------------------------------------------- 28 | ERROR | [x] There must be no blank lines after the function comment 31 | ERROR | [x] Expected 1 blank line before function; 2 found ----------------------------------------------------------------------------------------------------------------------------------------------------------- PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY ----------------------------------------------------------------------------------------------------------------------------------------------------------- FILE: /web/modules/custom/ckeditor_infocard/README.md ---------------------------------------------------------------------------------------------------------------- FOUND 0 ERRORS AND 7 WARNINGS AFFECTING 7 LINES ---------------------------------------------------------------------------------------------------------------- 15 | WARNING | Line exceeds 80 characters; contains 144 characters 50 | WARNING | Line exceeds 80 characters; contains 111 characters 57 | WARNING | Line exceeds 80 characters; contains 98 characters 59 | WARNING | Line exceeds 80 characters; contains 108 characters 60 | WARNING | Line exceeds 80 characters; contains 87 characters 61 | WARNING | Line exceeds 80 characters; contains 97 characters 62 | WARNING | Line exceeds 80 characters; contains 86 characters ---------------------------------------------------------------------------------------------------------------- FILE: /web/modules/custom/ckeditor_infocard/ckeditor_infocard.module ------------------------------------------------------------------------------------------------------------------------------- FOUND 1 ERROR AFFECTING 1 LINE ------------------------------------------------------------------------------------------------------------------------------- 34 | ERROR | [x] Expected 1 blank line after function; 0 found ------------------------------------------------------------------------------------------------------------------------------- PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY ------------------------------------------------------------------------------------------------------------------------------- FILE: /web/modules/custom/ckeditor_infocard/ckeditor_infocard.info.yml --------------------------------------------------------------------------------------------------------------------------------- FOUND 0 ERRORS AND 3 WARNINGS AFFECTING 1 LINE --------------------------------------------------------------------------------------------------------------------------------- 1 | WARNING | Remove "project" from the info file, it will be added by drupal.org packaging automatically 1 | WARNING | Remove "datestamp" from the info file, it will be added by drupal.org packaging automatically 1 | WARNING | Remove "version" from the info file, it will be added by drupal.org packaging automatically --------------------------------------------------------------------------------------------------------------------------------- FILE: /web/modules/custom/ckeditor_infocard/tests/src/FunctionalJavascript/CKEditorInfoCardTest.php -------------------------------------------------------------------------------------------------------------------------------------------------------------- FOUND 1 ERROR AND 3 WARNINGS AFFECTING 4 LINES -------------------------------------------------------------------------------------------------------------------------------------------------------------- 110 | ERROR | [x] Line indented incorrectly; expected 4 spaces, found 6 191 | WARNING | [ ] Line exceeds 80 characters; contains 81 characters 242 | WARNING | [ ] Line exceeds 80 characters; contains 82 characters 297 | WARNING | [ ] Line exceeds 80 characters; contains 82 characters -------------------------------------------------------------------------------------------------------------------------------------------------------------- PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY -------------------------------------------------------------------------------------------------------------------------------------------------------------- Time: 147ms; Memory: 12MB

Comments & Activities

Issue created by @dev2.addweb
Merge request !1#3458698 fixed phpcs issues → (Merged) created by Unnamed author
Pipeline finished with Failed
5 months ago
Total: 42s
#214555
Issue was unassigned.
Status changed to Needs review 5 months ago8:00am 3 July 2024
Comment 5 months ago →
🇮🇳India dev2.addweb
Hi,
I have fixed phpcs issues and created MR. Please review it.

Comment 4 months ago →

🇮🇳India bhaveshdas

i have tested MR 1 and i have found some error below:-
use below this command-: vendor/bin/phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig modules/custom/ckeditor_infocard-3458698/

FILE: /var/www/html/drupal-contri-d10/modules/custom/ckeditor_infocard-3458698/css/infocard.frontend.css
--------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
--------------------------------------------------------------------------------------------------------
 33 | ERROR | [x] CSS colours must be defined in lowercase; expected #f5f5f5 but found #F5F5F5
--------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------------------------------

Time: 1.53 secs; Memory: 12MB

Status changed to Needs work 4 months ago6:01am 4 July 2024
Comment 4 months ago →
🇮🇳India bhaveshdas
Pipeline finished with Failed
4 months ago
Total: 41s
#215394
Status changed to Needs review 4 months ago6:43am 4 July 2024
Comment 4 months ago →
🇮🇳India dev2.addweb
Hi,
I have fixed phpcs issue as mention in #4. Please review it.
Comment 4 months ago →
System Message

sianto → committed 5218d231 on 1.0.x authored by nilesh.addweb →
#3458698 fixed phpcs issues
Status changed to Fixed 4 months ago6:46am 5 July 2024
Comment 4 months ago →
sianto
Comment 4 months ago →
🇮🇳India dev2.addweb
@sianto, when you move an issue to the "Fixed" state, please remember to give credit to the contributors.
Comment 4 months ago →
Dannyjanse
Comment 4 months ago →
System Message
Automatically closed - issue fixed for 2 weeks with no activity.

Problem/Motivation

Merge Requests

!1Fix phpcs issues
Merged

Comments & Activities

Fix phpcs issues

Problem/Motivation

Merge Requests

!1Fix phpcs issuesMerged

Comments & Activities

!1Fix phpcs issues
Merged