Problem/Motivation

Drupal Coding Standards Issues | phpcs

Steps to reproduce

FILE: /taxonomy_delete/taxonomy_delete.drush.inc
-------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
-------------------------------------------------------------------------------------------
31 | ERROR | [x] Whitespace found at end of line
-------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-------------------------------------------------------------------------------------------

FILE: /taxonomy_delete/src/Form/TaxonomyDeleteUIForm.php
-----------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
-----------------------------------------------------------------------------------------------------------------
12 | WARNING | The class short comment should describe what the class does and not simply repeat the class name
-----------------------------------------------------------------------------------------------------------------

FILE: /taxonomy_delete/src/TermDelete.php
---------------------------------------------------------------------------------------------------------------------
FOUND 4 ERRORS AND 1 WARNING AFFECTING 5 LINES
---------------------------------------------------------------------------------------------------------------------
3 | ERROR | [x] There must be one blank line after the namespace declaration
10 | WARNING | [ ] The class short comment should describe what the class does and not simply repeat the class name
12 | ERROR | [x] Whitespace found at end of line
19 | ERROR | [x] Whitespace found at end of line
26 | ERROR | [x] Whitespace found at end of line
---------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 4 MARKED SNIFF VIOLATIONS AUTOMATICALLY
---------------------------------------------------------------------------------------------------------------------

CommentFileSizeAuthor
#2 coding-standard-3341465-2.patch1.63 KBomkar_yewale
Command icon 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:

Comments

omkar_yewale created an issue. See original summary.

omkar_yewale’s picture

Assigned: omkar_yewale » Unassigned
Status: Active » Needs review
StatusFileSize
new1.63 KB

Please Review this Patch.

sahil.goyal made their first commit to this issue’s fork.

sahil.goyal’s picture

Status: Needs review » Reviewed & tested by the community
Issue tags: +Drupal Coding Standards Issues | phpcs

Applied the #2 resolved all the php coding standard Issues/warnings, creating MR
moving to RTBC, Thanks.

abelcain’s picture

Dear @sahil.goyal,

I did a review of phpcs and phpmd in the issue https://www.drupal.org/project/taxonomy_delete/issues/3341043

If you agree, we can focus on that review, I can give you contribution to the commits of phpmd and phpcs. Just help me review that ticket and move forward so we have module ported to D10 and with correction of standards.

If it's ok to you, we can close this ticket as duplicated, and you can get the Port to D10 issue to review it.

Kind regards
Abel S.

AstonVictor made their first commit to this issue’s fork.

astonvictor’s picture

Status: Reviewed & tested by the community » Fixed
Issue tags: -

The MR is merged. Fixed all CS issues.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.