Skip to content

Conversation

@peterwilsoncc
Copy link
Collaborator

@peterwilsoncc peterwilsoncc commented Dec 18, 2025

All Submissions:

  • Does your code follow the WooCommerce Sniffs variant of WordPress coding standards?
  • Have you written new tests for your changes, as applicable?
  • Have you successfully run tests with your changes locally?
  • Will this change require new documentation or changes to existing documentation?

Changes proposed in this Pull Request:

Removes the application of the status: e2e tests ... labels during the E2E tests.

These were used in a private repo to limit E2E tests to run on demand, with the switch to the public repo the E2E test suite was reconfigured to run on each pull request.

Closes https://linear.app/a8c/issue/SQUARE-199/actions-remove-e2e-test-status-badges

Steps to test the changes in this Pull Request:

  1. Ensure the E2E tests have not added the status: e2e tests... labels to this pull request
  2. There is no step two.

Changelog entry

Dev - Remove application of E2E status labels from GitHub actions.

@peterwilsoncc peterwilsoncc self-assigned this Dec 18, 2025
@peterwilsoncc
Copy link
Collaborator Author

@iamdharmesh I know we usually merge in bulk in the lead up to a release but do you think this can be merged prior as it doesn't contain a source code change?

@iamdharmesh
Copy link
Collaborator

@iamdharmesh I know we usually merge in bulk in the lead up to a release but do you think this can be merged prior as it doesn't contain a source code change?

Sorry, somehow I missed this notification last year, but yes we are good to merge this. cc: @vikrampm1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants