Problem/Motivation

Saving time on checking standards and practice before committing any code.


Drupal coding standard and Drupal Practice check should pass

PHPCS, PHPCBF for both --standard=Drupal and --standard=DrupalPractice

Check Varbase Workflow Drupal coding standard

phpcbf --standard=Drupal --extensions=php,inc,install,test,profile,theme,scss,css,info,txt,md,yml /var/www/html/products/varbase_workflow/ --ignore=/var/www/html/products/varbase_workflow/node_modules/

phpcs --standard=Drupal --extensions=php,module,inc,install,test,theme,scss,css,info,txt,md,yml /var/www/html/products/varbase_workflow/ --ignore=/var/www/html/products/varbase_workflow/node_modules/

Check Varbase Workflow Drupal Practice

phpcbf --standard=DrupalPractice --extensions=php,module,inc,install,test,theme,scss,css,info,txt,md,yml /var/www/html/products/varbase_workflow/ --ignore=/var/www/html/products/varbase_workflow/node_modules/

phpcs --standard=DrupalPractice --extensions=php,module,inc,install,test,theme,scss,css,info,txt,md,yml /var/www/html/products/varbase_workflow/ --ignore=/var/www/html/products/varbase_workflow/node_modules/

Proposed resolution

yarn phpcs
yarn phpcbf

as

  "scripts": {
    "phpcs": "phpcs --standard=./.phpcs.xml .",
    "phpcbf": "phpcbf --standard=./.phpcs.xml ."
  },

Comments

RajabNatshah created an issue. See original summary.

  • RajabNatshah committed a05a340 on 2.0.x
    Issue #3175526: Add yarn script commands for Drupal coding standard and...
Rajab Natshah’s picture

Rajab Natshah’s picture

Assigned: Rajab Natshah » Mohammed J. Razem
Status: Active » Needs review
Issue tags: +varbase-8.8.7, +varbase-9.0.0
Rajab Natshah’s picture

Assigned: Mohammed J. Razem » Unassigned
Rajab Natshah’s picture

Status: Needs review » Fixed
Rajab Natshah’s picture

Rajab Natshah’s picture

Status: Fixed » Closed (fixed)

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