Skip to content
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

fix(ci): analyses snapshot test trigger and step if logic improvements #15576

Merged
merged 1 commit into from
Jul 3, 2024

Conversation

y3rsh
Copy link
Collaborator

@y3rsh y3rsh commented Jul 3, 2024

Update analyses snapshot workflow so that:

  • doc or test changes alone do not trigger the test
  • only run the snapshot-update if the tests actually ran and failed
  • only run the PR creator if the snapshot update actually ran successfully
  • only comment on the PR if the PR creator was successful
  • update to v6 of PR creator

@y3rsh y3rsh self-assigned this Jul 3, 2024
@y3rsh y3rsh requested a review from a team as a code owner July 3, 2024 18:20
Copy link
Contributor

@SyntaxColoring SyntaxColoring left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TY!

@y3rsh y3rsh merged commit 34d92b7 into edge Jul 3, 2024
10 checks passed
@y3rsh y3rsh deleted the stricter-analyses-test-trigger branch July 3, 2024 19:08
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.

2 participants