Skip to content

ci/github-script/commits: clarify commit hash not found message#466088

Merged
wolfgangwalther merged 1 commit into
NixOS:masterfrom
mdaniels5757:ci-update-commit-hash-not-found-message
Nov 29, 2025
Merged

ci/github-script/commits: clarify commit hash not found message#466088
wolfgangwalther merged 1 commit into
NixOS:masterfrom
mdaniels5757:ci-update-commit-hash-not-found-message

Conversation

@mdaniels5757

@mdaniels5757 mdaniels5757 commented Nov 28, 2025

Copy link
Copy Markdown
Member

The old message was confusing in (at least) #466009, see Matrix.

Things done

  • Built on platform:
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • Tested, as applicable:
  • Ran nixpkgs-review on this PR. See nixpkgs-review usage.
  • Tested basic functionality of all binary files, usually in ./result/bin/.
  • Nixpkgs Release Notes
    • Package update: when the change is major or breaking.
  • NixOS Release Notes
    • Module addition: when adding a new NixOS module.
    • Module update: when the change is significant.
  • Fits CONTRIBUTING.md, pkgs/README.md, maintainers/README.md and other READMEs.

Add a 👍 reaction to pull requests you find important.

@nixpkgs-ci nixpkgs-ci 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. 6.topic: continuous integration Affects continuous integration (CI) in Nixpkgs, including Ofborg and GitHub Actions backport release-25.05 labels Nov 28, 2025
@nixpkgs-ci nixpkgs-ci Bot added the 12.approvals: 2 This PR was reviewed and approved by two persons. label Nov 29, 2025
@wolfgangwalther wolfgangwalther added this pull request to the merge queue Nov 29, 2025
@nixpkgs-ci nixpkgs-ci Bot added 12.approvals: 3+ This PR was reviewed and approved by three or more persons. and removed 12.approvals: 2 This PR was reviewed and approved by two persons. labels Nov 29, 2025
Merged via the queue into NixOS:master with commit 8b8223e Nov 29, 2025
35 of 38 checks passed
@nixpkgs-ci

nixpkgs-ci Bot commented Nov 29, 2025

Copy link
Copy Markdown
Contributor

Successfully created backport PR for release-25.05:

@nixpkgs-ci nixpkgs-ci Bot mentioned this pull request Nov 29, 2025
Merged
@nixpkgs-ci

nixpkgs-ci Bot commented Nov 29, 2025

Copy link
Copy Markdown
Contributor

Successfully created backport PR for release-25.11:

@nixpkgs-ci nixpkgs-ci Bot mentioned this pull request Nov 29, 2025
Merged
@github-actions github-actions Bot added the 8.has: port to stable This PR already has a backport to the stable release. label Nov 29, 2025
@mdaniels5757 mdaniels5757 deleted the ci-update-commit-hash-not-found-message branch November 29, 2025 15:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@MattSturgeon MattSturgeon MattSturgeon approved these changes
@wolfgangwalther wolfgangwalther wolfgangwalther approved these changes
@Mic92 Mic92 Awaiting requested review from Mic92
@philiptaron philiptaron Awaiting requested review from philiptaron
@zowoq zowoq Awaiting requested review from zowoq

Assignees

No one assigned

Labels

6.topic: continuous integration Affects continuous integration (CI) in Nixpkgs, including Ofborg and GitHub Actions 8.has: port to stable This PR already has a backport to the stable release. 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. 12.approvals: 3+ This PR was reviewed and approved by three or more persons.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@mdaniels5757 @MattSturgeon @wolfgangwalther