Skip to content

Try/GitHub actions #594

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

Closed
desrosj wants to merge 179 commits into from
Closed

Try/GitHub actions #594

Changes from 1 commit
Commits
Show all changes
179 commits
Select commit Hold shift + click to select a range
c1e06e5
Experimenting with GitHub Actions.
desrosj Sep 29, 2020
2f99e0b
Run `npm install` and `npm run grunt`.
desrosj Sep 29, 2020
57e88b7
Try v2 of `actions/setup-node`.
desrosj Sep 29, 2020
9f94b1a
Remove `actions/setup-node` and use `nvm` to install correct version …
desrosj Sep 29, 2020
0362939
Start adding PHP specific test jobs.
desrosj Sep 29, 2020
bf8954d
Fix expected syntax.
desrosj Sep 29, 2020
7fc31db
Each job requires a `runs-on`.
desrosj Sep 29, 2020
c422215
Results of jobs don't get shared.
desrosj Sep 29, 2020
3669b9a
Remove job dependencies.
desrosj Sep 29, 2020
5b1db04
Add a delay so the MySQL container finishes setting up.
desrosj Sep 29, 2020
fe9667f
Re-arrange steps a bit.
desrosj Sep 29, 2020
8d0a8a3
Add a job for outputting test info
desrosj Sep 29, 2020
18841bb
Update tests.
desrosj Sep 29, 2020
b36cfb1
Try to fix permission issue within Docker container.
desrosj Sep 29, 2020
b771736
More tests.
desrosj Sep 30, 2020
8f59348
Checkout the WordPress Importer.
desrosj Sep 30, 2020
4111e82
Move some settings to `defaults`.
desrosj Sep 30, 2020
b1546fb
Fix nesting.
desrosj Sep 30, 2020
a893dec
Can't use `runs-on` within `defaults`
desrosj Sep 30, 2020
9e04d5b
Try using newer version of Docker Compose
desrosj Sep 30, 2020
ec3b8e6
Use setup-node action instead
desrosj Sep 30, 2020
c15cd2e
Store built WordPress as an artifact.
desrosj Sep 30, 2020
4842815
Fix yml syntax.
desrosj Sep 30, 2020
f4fdc39
Look at contents after checking out repo.
desrosj Sep 30, 2020
1c1c8f9
Don't include node_modules in artifacts.
desrosj Sep 30, 2020
b965cbe
Add additional job to test artifact usage.
desrosj Sep 30, 2020
f4f8fa9
Attempt to use caching for NPM
desrosj Sep 30, 2020
70b04c9
Fix syntax issues.
desrosj Sep 30, 2020
a17e84b
Missing install command.
desrosj Sep 30, 2020
9958b6c
Install first, duh.
desrosj Sep 30, 2020
e6bda0f
Attempt to use NPM cache and log `pwd`.
desrosj Sep 30, 2020
251a54e
Try using a different directory for GitHub actions.
desrosj Sep 30, 2020
aaa1703
Strip some stuff out.
desrosj Sep 30, 2020
86b37fb
Add a `which npm`.
desrosj Sep 30, 2020
9abc79c
Add more debug output.
desrosj Sep 30, 2020
d9ae6dc
Add more debug info
desrosj Sep 30, 2020
ff82587
Update PHP syntax.
desrosj Sep 30, 2020
ed0927c
More debug info.
desrosj Sep 30, 2020
a9fa48f
More debug info.
desrosj Sep 30, 2020
0d14cdc
Merge branch 'master' into try/github-actions
desrosj Oct 7, 2020
15b33c8
Update with `trunk` and remove failing command.
desrosj Oct 7, 2020
0f180e9
Try removing specific PHP version environment variable.
desrosj Oct 7, 2020
cc72536
See if the actions/checkout action causes permission issues.
desrosj Oct 7, 2020
a9c7eb1
Fix syntax error.
desrosj Oct 7, 2020
63d6deb
Fix syntax error.
desrosj Oct 7, 2020
9e418d1
Check the running Docker containers throughout.
desrosj Oct 7, 2020
e204fdf
Check directory moving up a directory has different permissions.
desrosj Oct 7, 2020
6c2ed36
Test matrixes, and revert last change.
desrosj Oct 7, 2020
546d618
Add missing runs on.
desrosj Oct 7, 2020
090bdb0
Check permissions
desrosj Oct 7, 2020
6523fb5
Check permissions
desrosj Oct 7, 2020
c3c751e
Try using `--allow-root`.
desrosj Oct 7, 2020
55c8354
Define some PHP env vars.
desrosj Oct 7, 2020
8f0e112
Remove test command.
desrosj Oct 7, 2020
a11a71b
Get the code to run.
desrosj Oct 7, 2020
60ad740
Need to set PHP_FPM_UID
desrosj Oct 7, 2020
1823818
Remove env var from global.
desrosj Oct 7, 2020
f9dafc5
Try again
desrosj Oct 7, 2020
0d1d0c8
Flip
desrosj Oct 7, 2020
c043e85
This time?
desrosj Oct 7, 2020
787840f
Now?
desrosj Oct 7, 2020
2cd36d0
Reverse?
desrosj Oct 7, 2020
3953261
Use the right var name.
desrosj Oct 7, 2020
e6260b4
🙌🏼
desrosj Oct 7, 2020
170a020
Prevent `.0` from being trimmed.
desrosj Oct 7, 2020
339accb
PHP 8 needs to use the `test:php-composer` command.
desrosj Oct 7, 2020
0010dbe
Add missing `=`.
desrosj Oct 7, 2020
69a6142
Use the right LOCAL_PHP value.
desrosj Oct 8, 2020
33f2ea6
Add composer install step.
desrosj Oct 8, 2020
5d84647
Check for true.
desrosj Oct 8, 2020
b4fac11
Add Windows testing
desrosj Oct 8, 2020
1ce74e8
Add caching of node_modules and artifact uploading.
desrosj Oct 8, 2020
c278372
Fix action syntax.
desrosj Oct 8, 2020
9af2938
Fix nvmrc detection.
desrosj Oct 8, 2020
feae723
Don't worry about nvmrc right now.
desrosj Oct 8, 2020
8da05d2
Use correct action name.
desrosj Oct 8, 2020
e17168d
Try zipping the artifact for speed.
desrosj Oct 8, 2020
b04d40b
Fix typo.
desrosj Oct 8, 2020
d449e53
Don't need to build WP in the test action.
desrosj Oct 8, 2020
8434eee
ZIP is not available for Windows currently.
desrosj Oct 8, 2020
48abada
Refine zipping.unzipping.
desrosj Oct 8, 2020
e5c80f4
Log UID and GID
desrosj Oct 8, 2020
d3bbd8a
Some debug info.
desrosj Oct 8, 2020
24d587e
Build doesn't exist yet.
desrosj Oct 8, 2020
d9df2bd
Correct file ownership after unzipping the artifact.
desrosj Oct 8, 2020
ef4815f
Correct variable name.
desrosj Oct 8, 2020
64c112b
Try a different unzipping.
desrosj Oct 8, 2020
97867f6
Don't use an action to unzip.
desrosj Oct 8, 2020
103b6eb
Add inline documentation for each job.
desrosj Oct 8, 2020
7ba0d12
Fix syntax error.
desrosj Oct 8, 2020
c2e3eca
no message
desrosj Oct 8, 2020
52de448
Close else if correctly
desrosj Oct 8, 2020
9eebef2
Fix syntax error.
desrosj Oct 8, 2020
7a65b51
Chromium is required for JS tests.
desrosj Oct 8, 2020
1018e13
Unset Puppeteer correctly.
desrosj Oct 8, 2020
8071492
Add a memcached test job.
desrosj Oct 8, 2020
f6c377c
Add PHP compatibility testing job.
desrosj Oct 8, 2020
b74e237
Fix conditional expression checking for the memcached job.
desrosj Oct 8, 2020
8a15053
Move PHP compatibility testing to it's own workflow.
desrosj Oct 8, 2020
be3dfd4
Fix conditional checking for memcached.
desrosj Oct 8, 2020
483ea16
Only start the memcached server if it's wanted.
desrosj Oct 8, 2020
2d4214d
Add an action for the test reporter.
desrosj Oct 8, 2020
c8e07c5
Fix syntax error.
desrosj Oct 8, 2020
f73ab26
Fix test runner repo checkout.
desrosj Oct 8, 2020
189958f
Add caching to Composer.
desrosj Oct 8, 2020
99a63f9
Always cache composer for PHP compatibility testing.
desrosj Oct 8, 2020
e7ce903
Try using the PHPCS action
desrosj Oct 9, 2020
f00d615
Use old method.
desrosj Oct 9, 2020
aaa6df5
Add a workflow for coding standards checks.
desrosj Oct 9, 2020
48ddf70
Run coding standards checks on push for now.
desrosj Oct 9, 2020
e6a3c28
When a new commit is made, cancel previous jobs that are not completed.
desrosj Oct 9, 2020
7a65511
Fix workflow error.
desrosj Oct 9, 2020
7c9565f
Another syntax error.
desrosj Oct 9, 2020
a694d1a
Fix coding standards syntax error.
desrosj Oct 9, 2020
6c2b878
Allow all linters to complete if one returns an error.
desrosj Oct 9, 2020
b9e9a51
Try passing path to phpcs
desrosj Oct 9, 2020
7c80b48
Don't setup PHP
desrosj Oct 9, 2020
be43648
Add PHPCS
desrosj Oct 9, 2020
c1af97e
Debug composer.
desrosj Oct 9, 2020
623ff43
Devbug composer issue.
desrosj Oct 9, 2020
db7197b
Try disabling cache.
desrosj Oct 9, 2020
d65ce9b
Remove composer diagnose.
desrosj Oct 9, 2020
d50d41e
Look for phpcs
desrosj Oct 9, 2020
18d62be
Try global install
desrosj Oct 9, 2020
fb686ed
Debug info
desrosj Oct 9, 2020
099174a
no message
desrosj Oct 9, 2020
f9a3f29
Remove debug
desrosj Oct 9, 2020
0c3b26a
Specify shell
desrosj Oct 9, 2020
96e8d06
This should work!
desrosj Oct 9, 2020
d23cf34
Use new syntax.
desrosj Oct 9, 2020
0576d6e
Add reporting to code standards checks.
desrosj Oct 9, 2020
6c031dd
Remove the `--report` flag. The action runs with `-q` to ensure valid…
desrosj Oct 9, 2020
d00cd73
Update the compatibility testing workflow to have annotations and not…
desrosj Oct 9, 2020
746417f
Cause PHP compatibility and coding standards issues.
desrosj Oct 9, 2020
df7eb7c
Explicitly pass standard to the test command.
desrosj Oct 9, 2020
d9fe93b
Clean up and document.
desrosj Oct 9, 2020
c820818
Add end to end test workflow. Clean up inline docs.
desrosj Oct 9, 2020
3278863
Add PHPCompat error.
desrosj Oct 9, 2020
b972a30
Test caching PHPCS runs.
desrosj Oct 9, 2020
5df7374
Purge the cache.
desrosj Oct 9, 2020
a0f2f15
Try adding a tests specific scan.
desrosj Oct 9, 2020
d87aacc
Remove test.
desrosj Oct 9, 2020
e7d1ceb
Don't skip Puppeteer on E2E
desrosj Oct 9, 2020
c6ffc5d
Disable composer caching.
desrosj Oct 9, 2020
346edc9
Why is php compat failing
desrosj Oct 9, 2020
c172d5a
Add environment variables.
desrosj Oct 9, 2020
12aab9c
Fix order of composer install
desrosj Oct 9, 2020
703972a
Back to using the action.
desrosj Oct 9, 2020
9a0b75e
Try different path to PHPCS config.
desrosj Oct 9, 2020
e7f6109
no message
desrosj Oct 9, 2020
be59699
Update test-wordpress.yml
desrosj Oct 9, 2020
5c87d55
Update test-wordpress.yml
desrosj Oct 9, 2020
569ba44
Fix conditionals
desrosj Oct 12, 2020
621d2c0
Fix syntax error.
desrosj Oct 12, 2020
5e4ed34
Use the matrix to detect memcached
desrosj Oct 12, 2020
d1bcb42
Exclude the tests directory in the ruleset since `phpcs` command is h…
desrosj Oct 12, 2020
e023577
Clean up documentation. Add explicit exclusion for the `tests` direct…
desrosj Oct 12, 2020
75f12ae
Move JavaScript tests to their own workflow.
desrosj Oct 12, 2020
9c8faf4
Don't need to log PHP debug information in the JavaScript workflow.
desrosj Oct 12, 2020
ca712e7
Limit PHPCS debug info.
desrosj Oct 12, 2020
06a9a6e
Fix name of the `test-js` job.
desrosj Oct 12, 2020
fb762f7
Naming adjustments.
desrosj Oct 12, 2020
410e841
Updat docs.
desrosj Oct 12, 2020
3f2267d
Indicate that the JS standards checks are run in this workflow.
desrosj Oct 12, 2020
d39f0ab
Add a workflow that welcomes new contributors.
desrosj Oct 13, 2020
429d29f
Add Windows to see how that goes.
desrosj Oct 13, 2020
1ce0fd5
Clarify NPM/NodeJS docs.
desrosj Oct 13, 2020
09d1064
Log some docker debug info before attempting to use commands.
desrosj Oct 13, 2020
6731e17
The `bridge` driver is not available on Windows.
desrosj Oct 13, 2020
1b20469
Trying to fix comments on PRs from forks.
desrosj Oct 13, 2020
96c10e7
Remove Windows testing for now.
desrosj Oct 13, 2020
cbc31d8
Add a workflow for testing `npm` commands on Windows.
desrosj Oct 13, 2020
5686c78
Update the name of the welcome action.
desrosj Oct 13, 2020
d706281
Add `check_name` to prevent results from posting on different workflo…
desrosj Oct 13, 2020
c233838
Add cron runs for each version branch. Also, update `on` configuration.
desrosj Oct 13, 2020
7f5252c
Fix syntax error.
desrosj Oct 13, 2020
4fcd248
Do not continue on error.
desrosj Oct 13, 2020
95c4622
Fix syntax error.
desrosj Oct 13, 2020
7e84c26
Revert "Do not continue on error."
desrosj Oct 13, 2020
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
Look for phpcs
  • Loading branch information
@desrosj
desrosj committed Oct 9, 2020
commit d50d41e80e9ebc49dd74553bb43c9f53903dea28
4 changes: 3 additions & 1 deletion .github/workflows/coding-standards.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,9 @@ jobs:
php --version

- name: Install Composer dependencies
run: composer install
run: |
composer install
phpcs -v

- name: Run PHPCS on WordPress Core files
uses: wearerequired/lint-action@v1
Expand Down