Skip to content

Commit d3e874d

Browse files
richardbporterRichard B. Porter
andauthored
Run drupal-check in CI and use Drupal PHPCS since we use Drupal APIs. (#67)
* Require mglaman/drupal-check, symfony/phpunit-bridge dev deps * Ignore ds store * Run drupal-check in CI * Specify drupal-check dirs instead of excludes * Fix Datetime case * Remove incorrect @return doc * Require dev drupal/coder * Run drupal PHPCS on CI * PHPCBF * Implement Drupal coding standards Co-authored-by: Richard B. Porter <rbp@richardbporter.com>
1 parent 23b8fb1 commit d3e874d

File tree

8 files changed

+1437
-501
lines changed

8 files changed

+1437
-501
lines changed

.github/workflows/ci.yml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,8 @@ jobs:
2929
- name: Install drupal
3030
run: ./vendor/bin/drush site:install testing --root sut --db-url mysql://root:root@localhost/drupal --yes
3131
- name: Run code sniffer
32-
run: ./vendor/bin/phpcs --standard=PSR2 --ignore='sut,vendor' .
32+
run: ./vendor/bin/phpcs --standard=Drupal,DrupalPractice --ignore='sut,vendor' .
33+
- name: Run drupal-check
34+
run: ./vendor/bin/drupal-check UsersCommands.php tests/
3335
- name: Run unit tests
3436
run: ./vendor/bin/phpunit

.gitignore

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,4 +3,5 @@ vendor
33
.vscode
44
.scenarios.lock
55
sut
6-
.phpunit.result.cache
6+
.phpunit.result.cache
7+
.DS_Store

0 commit comments

Comments
 (0)