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.
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
Update workflow to use latest checkout actions #45199
Conversation
Hi , @woocommerce/vortex Apart from reviewing the code changes, please make sure to review the testing instructions as well. You can follow this guide to find out what good testing instructions should look like: |
Test Results SummaryCommit SHA: bbcc650
To view the full API test report, click here. To view the full E2E test report, click here. To view all test reports, visit the WooCommerce Test Reports Dashboard. |
@@ -23,6 +23,7 @@ const config = { | |||
retries: CI ? 2 : 0, | |||
repeatEach: REPEAT_EACH ? Number( REPEAT_EACH ) : 1, | |||
workers: 1, | |||
reportSlowTests: { max: 5, threshold: 30 * 1000 }, // 30 seconds threshold |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I picked some somewhat arbitrary numbers here, we can adjust. What this says is that a test will be reported as slow if it takes longer than 30 seconds. The max number is that it's reporting the 5 slowest tests per shard... having a list of more than 25 tests probably isn't super productive.
This PR is mostly geared towards cleaning up the annotations on test runs: https://github.com/woocommerce/woocommerce/actions/runs/8101450263?pr=45199 It eliminates some deprecation warnings and tries to limit the number of slow tests reported to a manageable number. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work @lanej0.
* Update workflow to use latest checkout actions * Update non-HPOS workflow as well * Update additional actions * Set a slow test threshold at 30 seconds (15 is default) * Final action updates * Slow tests has to be an object * Pin external actions to commit * Slow test threshold and max --------- Co-authored-by: Jon Lane <jon.lane@automattic.com>
Submission Review Guidelines:
Changes proposed in this Pull Request:
Updates the PR testing workflow to eliminate the GitHub warnings for deprecated Node.js use:
How to test the changes in this Pull Request:
Using the WooCommerce Testing Instructions Guide, include your detailed testing instructions:
Changelog entry
Significance
Type
Message
Comment