Problem/Motivation

It looks like this module is getting a lot of traction and lots should be happening soon. Let's fix the coding standard while it's still "small" :)

Proposed resolution

  1. Add a phpcs.xml.dist to the project, loosely based on the one we added to jsonapi here #2801919: [FEATURE] Fix coding standards with phpcbf.
  2. Fix the coding standards.

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#2 feature_fix_coding-2875309-2.patch9.98 KBshadcn
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

arshadcn created an issue. See original summary.

shadcn’s picture

Status: Active » Needs review
FileSize
9.98 KB
e0ipso’s picture

+1

e0ipso’s picture

Status: Needs review » Fixed

This actually highlights that this was some code that I put together in a rush to be able to share it with other people.

  • e0ipso committed e252b51 on 8.x-1.x authored by arshadcn
    docs(CS): Fix coding standards (#2875309 by arshadcn)
    
shadcn’s picture

@e0ipso And thanks so much for sharing :)

Status: Fixed » Closed (fixed)

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