Skip to content

[Backport release-24.11] workflows/no-channel: run again when base changed#404464

Merged
wolfgangwalther merged 1 commit intorelease-24.11from
backport-404461-to-release-24.11
May 5, 2025
Merged

[Backport release-24.11] workflows/no-channel: run again when base changed#404464
wolfgangwalther merged 1 commit intorelease-24.11from
backport-404461-to-release-24.11

Conversation

@nixpkgs-ci
Copy link
Copy Markdown
Contributor

@nixpkgs-ci nixpkgs-ci bot commented May 5, 2025

Bot-based backport to release-24.11, triggered by a label in #404461.

  • Before merging, ensure that this backport is acceptable for the release.
    • Even as a non-committer, if you find that it is not acceptable, leave a comment.

Because of the branches filter, the job would **not** re-run when only
the base-branch was changed, thus leaving the contributor with a failed
check.

With this change, the job should be triggered and then skipped in this
case, so the check in the list should be replaced with a skip.

Note, this also means that this step will show up in every PR as
skipped, even when it was never failed before.

(cherry picked from commit 40963bf)
@github-actions github-actions bot added 6.topic: continuous integration Affects continuous integration (CI) in Nixpkgs, including Ofborg and GitHub Actions 4.workflow: backport This targets a stable branch 6.topic: policy discussion Discuss policies to work in and around Nixpkgs labels May 5, 2025
@wolfgangwalther
Copy link
Copy Markdown
Contributor

(testing the change itself by changing the branch)

@wolfgangwalther wolfgangwalther changed the base branch from release-24.11 to nixpkgs-24.11-darwin May 5, 2025 19:56
@github-actions github-actions bot added 6.topic: python Python is a high-level, general-purpose programming language. 6.topic: php PHP is a general-purpose scripting language geared towards web development. 6.topic: k3s Kubernates distribution (https://k3s.io/) labels May 5, 2025
@wolfgangwalther wolfgangwalther changed the base branch from nixpkgs-24.11-darwin to release-24.11 May 5, 2025 19:57
@github-actions github-actions bot removed 6.topic: python Python is a high-level, general-purpose programming language. 6.topic: php PHP is a general-purpose scripting language geared towards web development. 6.topic: k3s Kubernates distribution (https://k3s.io/) labels May 5, 2025
@wolfgangwalther
Copy link
Copy Markdown
Contributor

OK, that test was stupid, because it would take the release-24.11 branch's state to (not) run the job. Thus, this just confirms the problem, the "No channel PR" is still there. Will test in a different PR against master for the difference.

@Mic92
Copy link
Copy Markdown
Member

Mic92 commented May 5, 2025

What about this one? #404443

@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 May 5, 2025
@wolfgangwalther
Copy link
Copy Markdown
Contributor

I confirmed it in #404466 - the check now appears as skipped.

@wolfgangwalther wolfgangwalther merged commit dbc573d into release-24.11 May 5, 2025
39 of 44 checks passed
@wolfgangwalther wolfgangwalther deleted the backport-404461-to-release-24.11 branch May 5, 2025 20:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4.workflow: backport This targets a stable branch 6.topic: continuous integration Affects continuous integration (CI) in Nixpkgs, including Ofborg and GitHub Actions 6.topic: policy discussion Discuss policies to work in and around Nixpkgs 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.

2 participants