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

Partially revert "build(deps): bump cachix/install-nix-action from 26 to 27" #329436

Merged
merged 1 commit into from
Jul 23, 2024

Conversation

infinisil
Copy link
Member

@infinisil infinisil commented Jul 23, 2024

Description of changes

This partially reverts #313109 because it's causing failures on master: https://github.com/NixOS/nixpkgs/actions/runs/10061768002/job/27812623186?pr=329433

I/O error:  Failed to get the definition info for attribute acl: In /home/runner/work/nixpkgs/nixpkgs/pkgs/stdenv/linux/default.nix, attribute parent node is not an attribute path node: [email protected]

The reason is because the pkgs/by-name check doesn't work for newer Nix versions yet, see NixOS/nixpkgs-vet#78

It's not enough to look at CI when updating GitHub workflows that trigger on pull_request_target, because they use the workflow file from the base branch, not the PR branch. Ping @Artturin @mweinelt


Add a 👍 reaction to pull requests you find important.

… to 27"

This partially reverts commit ab7becf.

The pkgs/by-name check doesn't work for newer Nix versions yet,
see NixOS/nixpkgs-vet#78
@infinisil
Copy link
Member Author

Reapplying this in #330297

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant