GitLab CI reports PHP_CodeSniffer errors/warnings which should be fixed.

Issue fork node_visibility-3475546

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:

About issue forks

Comments

sunil jolapara created an issue. See original summary.

sunil jolapara opened merge request !2

sunil jolapara’s picture

sunil jolapara commented
Status: Needs work » Needs review
avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Assigned: » Unassigned
Status: Needs review » Needs work
Issue tags: +Needs issue summary update

The issue summary for PHP_CodeSniffer issues should always show the report given from PHP_CodeSniffer and the arguments passed to phpcs.

Furthermore, since GitLab CI is now enabled for all the projects, GitLab CI should be used to get PHP_CodeSniffer errors/warnings.

zaryab_drupal’s picture

zaryab_drupal commented
Issue summary: View changes

avpaderno opened merge request !3

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Issue summary: View changes
Issue tags: -Needs issue summary update

avpaderno changed the visibility of the branch gitlab-ci-reports to hidden.

avpaderno changed the visibility of the branch 3475546-fix-the-issues-phpcodesniffer to hidden.

kul.pratap’s picture

kul.pratap commented
Assigned: Unassigned » kul.pratap

avpaderno opened merge request !4

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Assigned: kul.pratap » Unassigned
Status: Needs work » Needs review