Problem/Motivation

The module has issues with coding standards.

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

Yurii O. created an issue. See original summary.

yurii o.’s picture

pls check it

tmaiochi’s picture

Assigned: Unassigned » tmaiochi
Status: Active » Needs review

I'll review this.

gilmord’s picture

@Yurii O. patches is the old way to resolve the issues
please follow a new forks & MRs guide and create a merge request

thanks!

tmaiochi’s picture

Assigned: tmaiochi » Unassigned
StatusFileSize
new7.83 KB
new4.43 KB

The patch had some phpcs problems. I fixed some of them, but it has a warning about unused variable, however the variable is in using. So I left it at that and didn't change anything about this warning. I also sending the interdiff.

gilmord’s picture

@tmaiochi please use the forks & MRs flow mentioned a few comments above, so I can merge it.

victoria-marina’s picture

Assigned: Unassigned » victoria-marina

I'll review it.

victoria-marina’s picture

Assigned: victoria-marina » Unassigned
Status: Needs review » Reviewed & tested by the community

The #5 worked for me! RTBC

gilmord’s picture

Status: Reviewed & tested by the community » Fixed

Thanks all

Status: Fixed » Closed (fixed)

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