- Issue created by @shaimaa natshah
- 🇯🇴Jordan shaimaa natshah
Problem/Motivation
Drupal coding standard and Drupal Practice check should pass
PHPCS, PHPCBF for both --standard=Drupal and --standard=DrupalPractice
Check Web Security Drupal coding standard
phpcbf --standard=Drupal --extensions=php,module,inc,install,test,theme,scss,css,info,txt,md,yml /var/www/html/products/websecurity/ --ignore=/var/www/html/products/websecurity/node_modules/ phpcs --standard=Drupal --extensions=php,module,inc,install,test,theme,scss,css,info,txt,md,yml /var/www/html/products/websecurity/ --ignore=/var/www/html/products/websecurity/node_modules/
Check Web Security Drupal Practice
phpcbf --standard=DrupalPractice --extensions=php,module,inc,install,test,theme,scss,css,info,txt,md,yml /var/www/html/products/websecurity/ --ignore=/var/www/html/products/websecurity/node_modules/ phpcs --standard=DrupalPractice --extensions=php,module,inc,install,test,theme,scss,css,info,txt,md,yml /var/www/html/products/websecurity/ --ignore=/var/www/html/products/websecurity/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 ." },
-
Shaimaa Natshah →
committed 11bf8819 on 10.0.x
Issue #3379906: Add yarn script commands for Drupal coding standard and...
-
Shaimaa Natshah →
committed 11bf8819 on 10.0.x
- Issue was unassigned.
- Status changed to Fixed
over 1 year ago 9:19am 8 August 2023 Automatically closed - issue fixed for 2 weeks with no activity.