In preparation for security review, I'm running drupal-check and https://pareview.sh/.

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

joelsteidl created an issue. See original summary.

joelsteidl’s picture

StatusFileSize
new26.8 KB

This patch cleans up the majority of issues found by drupal-check.

  • joelsteidl committed 26a43d1 on 8.x-1.x
    Issue #3132308 by joelsteidl: Coding standards in preparation for...
joelsteidl’s picture

This will need a re-roll.

silvi.addweb made their first commit to this issue’s fork.

silvi.addweb’s picture

Status: Active » Needs review

Hello,

I created an MR and fixed phpcs problems. Review, please.

riddhi.addweb’s picture

Status: Needs review » Reviewed & tested by the community
StatusFileSize
new179.41 KB
new121.53 KB

I have tested this issue in version 8.9 and the mentioned phpcs issue is resolved, & I have also checked and it is working as expected. I am attaching the Screenshots & doing RTBC for the same.

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

lpeidro’s picture

  • I have added the GitLab CI configuration to run both static code analysis and unit tests in GitLab.
  • I have resolved some issues and added compatibility with Drupal 10.
  • I just merged it into the 2.x development branch.