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
Blocks E2E: Be more descriptive with the workflow titles #45381
Conversation
Hi @lanej0, @gigitux, @woocommerce/woo-fse 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: 9d163dc
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. |
0a3cc2c
to
6471626
Compare
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.
LGTM!
Changes proposed in this Pull Request:
Tweak the Blocks E2E job titles to make them more descriptive so they're easier to follow and debug:
Playwright E2E tests -
prefix as it's already aRun Blocks Playwright Tests
workflow.Normal
toDefault (Block) Theme
to reflect the env setup better.Classic
toClassic Theme
for the same reason.How to test the changes in this Pull Request:
Ensure the new titles are displayed correctly.