Problem/Motivation

Drupal coding standard and Drupal Practice check should pass

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

Check Webtheme Default Content Drupal coding standard

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

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

Check Webtheme Default Content Drupal Practice

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

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


Proposed resolution

yarn phpcs
yarn phpcbf
yarn lint:yaml

as

  "scripts": {
    "phpcs": "phpcs --standard=./.phpcs.xml .",
    "phpcbf": "phpcbf --standard=./.phpcs.xml .",
    "lint:yaml": "node ./node_modules/eslint/bin/eslint.js --config=.eslintrc.json --ext .yml ."
  },

Comments

Shaimaa Natshah created an issue. See original summary.

shaimaa natshah’s picture

shaimaa natshah’s picture

Issue summary: View changes

  • Shaimaa Natshah committed 6bfb867c on 10.0.x
    Issue #3382881: Add yarn script commands for Drupal coding standard and...
shaimaa natshah’s picture

Assigned: shaimaa natshah » webship
Status: Active » Needs review
shaimaa natshah’s picture

Assigned: webship » Unassigned
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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