-
Notifications
You must be signed in to change notification settings - Fork 115
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
[Tooling] Skip UI Tests if there are no relevant changes #15086
Conversation
55c2ddd
to
f9e23ef
Compare
|
f9e23ef
to
5f38416
Compare
52211d5
to
ef8f2b4
Compare
da74b28
to
a3de498
Compare
.buildkite/pipeline.yml
Outdated
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.
🎗️ Reminder to revert the changes on this file before undrafting this PR and merging it, since those are only for temporary debugging purposes.
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 updated the a8c toolkit plugin version, ran a couple of tests (see updated description) and reverted this temp change.
c603877
to
6c63f94
Compare
6c63f94
to
d7ed512
Compare
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
Documentation and Community
|
Description
This PR depends on Automattic/a8c-ci-toolkit-buildkite-plugin#148.
This PR updates the UI Test jobs to use the result of the command
pr_changed_files
froma8c-ci-toolkit-buildkite-plugin
to decide if tests should run or not.Testing