Problem/Motivation

Branch 8.x-1.1 has some coding standards messages.

Steps to reproduce

Run the phpcs command to know the coding standard suggestions

Proposed resolution

Need to apply the patch file to meet the coding standards

Remaining tasks

None

User interface changes

None

API changes

None

Data model changes

None

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

SivaprasadC created an issue. See original summary.

sivaprasadc’s picture

Assigned: sivaprasadc » Unassigned
Status: Active » Needs review
StatusFileSize
new10.51 KB

Attached patch should fix all coding standard messages.

sivaprasadc’s picture

Hi Team,

Before reviewing the patch, Kindly make sure you have applied this patch.

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

urvashi_vora’s picture

Assigned: Unassigned » urvashi_vora
Status: Needs review » Needs work
StatusFileSize
new63.76 KB

Hi,

I reviewed your patch. It failed.

The output is in the screenshot. Please check.

I am working on this ticket.

urvashi_vora’s picture

Assigned: urvashi_vora » Unassigned
Status: Needs work » Needs review
StatusFileSize
new11.68 KB
new710 bytes

Hi,

Please review this patch. Also providing interdiff file. But I got this error while creating interdiff.

urvasi@urvasi-Inspiron-15-3552:/var/www/html/contribution/d8_cont/web/modules/contrib/content_connected-3164826$ interdiff content_connected_coding_standard_fixes-3164826-2.patch coding-standards-resolved-3164826-6.patch > interdiff_2-6.txt
1 out of 1 hunk FAILED -- saving rejects to file /tmp/interdiff-1.TscJ0V.rej
interdiff: Error applying patch1 to reconstructed file

Thanks

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

sahil.goyal’s picture

Thanx @urvashi_vora changes look good, patch applies cleanly and not other PHP coding standard issues can be seen,
Creating MR.

Move to RTBC.

sahil.goyal’s picture

Status: Needs review » Reviewed & tested by the community
Issue tags: +Phpcs Drupal coding standard issue

bletch’s picture

Version: 8.x-1.1 » 2.1.1
Status: Reviewed & tested by the community » Fixed
Issue tags: -
bletch’s picture

Status: Fixed » Closed (fixed)