Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Coding standards v3 #70

Merged
merged 18 commits into from
Jul 10, 2024
Merged

Coding standards v3 #70

Changes from 1 commit
Commits
Show all changes
18 commits
Select commit Hold shift click to select a range
b4bef3b
Coding standards v3
danepowell Jul 1, 2024
63b4b46
fix standards
danepowell Jul 2, 2024
5e243bc
fix tests
danepowell Jul 2, 2024
1c884ee
Apply suggestions from code review
danepowell Jul 3, 2024
56f750d
remove rogue drupalpractice sniff
danepowell Jul 3, 2024
a426441
allow throws
danepowell Jul 3, 2024
80ac3a4
Grammar niggle.
TravisCarden Jul 8, 2024
9165867
alphabetize standards
danepowell Jul 8, 2024
d37f9ce
remove Edge
danepowell Jul 8, 2024
a229fa5
psr-12 comment comment
danepowell Jul 8, 2024
e8271b5
Spacing niggle.
TravisCarden Jul 9, 2024
e1dd263
Rename AcquiaPHP and AcquiaDrupal to AcquiaPHPMinimal and AcquiaDrupa…
danepowell Jul 9, 2024
5829fde
Add comment
danepowell Jul 9, 2024
3a9f620
allow package annotation
danepowell Jul 9, 2024
6eb2c86
Update Composer branch alias.
TravisCarden Jul 9, 2024
f7a666d
Fix missed artifact of renaming.
TravisCarden Jul 9, 2024
0ccaa75
Add PHP Compatibility sniffs to AcquiaDrupalMinimal.
TravisCarden Jul 9, 2024
2a73493
Revert "allow package annotation"
TravisCarden Jul 9, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix tests
  • Loading branch information
@danepowell
danepowell committed Jul 2, 2024
commit 5e243bc83fe11f168c7f347cecdb7bc53ddc9cb5
8 changes: 5 additions & 3 deletions .github/script.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 33,8 @@ cd ~/fixture || exit 1
INSTALLED=$(./vendor/bin/phpcs -i)
EXPECTED=(
AcquiaDrupalStrict
AcquiaDrupalTransitional
AcquiaDrupal
AcquiaPHPStrict
AcquiaPHP
Drupal
DrupalPractice
Expand All @@ -54,12 55,13 @@ if [[ "$FAILURES" ]]; then
fi

# Place a good test file.
printf "<?php\n\n/**\n * @file\n * Good test file.\n */\n" > good.php
printf "<?php\n\n/**\n * @file\n * Good test file.\n */\n\ndeclare(strict_types=1);\n" > good.php

# Test that the SUT's standards can be run.
EXPECTED=(
AcquiaDrupalStrict
AcquiaDrupalTransitional
AcquiaDrupal
AcquiaPHPStrict
AcquiaPHP
)
for STANDARD in "${EXPECTED[@]}"; do
Expand Down
Loading