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

actions/no-channel-pr: re run when base branch is updated #369179

Merged
merged 1 commit into from
Jan 3, 2025

Conversation

JohnRTitor
Copy link
Contributor

Additionally, no permissions are needed so set it to an empty set
Signed-off-by: John Titor [email protected]

Fixes a bug with this action reported in #355847 (comment)

Additionally, no permissions are needed so set it to an empty set
Signed-off-by: John Titor <[email protected]>
@github-actions github-actions bot added 6.topic: policy discussion 6.topic: continuous integration Affects continuous integration (CI) in Nixpkgs, including Ofborg and GitHub Actions labels Dec 29, 2024
@FliegendeWurst
Copy link
Member

I am not sure this will work. Wouldn't it just re-run the action and fail again?

@github-actions github-actions bot added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux labels Dec 29, 2024
@FliegendeWurst
Copy link
Member

In #369170, I tried adding some conditionals around github.base_ref - but it didn't really work since the value appears to be cached.

@JohnRTitor
Copy link
Contributor Author

JohnRTitor commented Dec 29, 2024

Nope, A manual re run is where some things are cached. This is an automatic re run, just like when actions are ran on a force push.

@FliegendeWurst
Copy link
Member

FliegendeWurst commented Dec 29, 2024

So, Github will:

  1. trigger the "edited" event
  2. notice the workflow no longer applies
  3. remove the previous failing workflow from the overview

In that case it should work. I am not familiar enough with GHA to say whether these steps are accurate however.

@JohnRTitor
Copy link
Contributor Author

Yep, on edited action it will restart the workflow on an entirely new set of variables, which is not the same as restarting a failed job.

@JohnRTitor
Copy link
Contributor Author

@ofborg eval

Copy link
Member

@infinisil infinisil left a comment

Choose a reason for hiding this comment

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

💯

@infinisil infinisil merged commit 0ac1641 into NixOS:master Jan 3, 2025
31 of 33 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6.topic: continuous integration Affects continuous integration (CI) in Nixpkgs, including Ofborg and GitHub Actions 6.topic: policy discussion 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants