Skip to content

Cannot use nixpkgs-review rev HEAD to check local commits added to an existing pull request #458

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

Open
dwt opened this issue Jan 28, 2025 · 2 comments

Comments

@dwt
Copy link

dwt commented Jan 28, 2025

This can be seen on NixOS/nixpkgs#366387 where I tried to checkout a PR do some local modifications and then tried to build it. That got me errors like this.

Now... did I do something wrong there or is this expected behavior? Quite possibly I do not understand correctly how it is supposed to work to improve pull requests from other people and make (tested) suggestions on how to improve them.

@dwt
Copy link
Author

dwt commented Apr 23, 2025

I am hitting this problem repeatedly. Basically whenever I have local changes to a nixpkgs pull request that I would like to test as a feedback to the pull request provider, I cannot seem to be able to do this with nix-review. Things I tried:

  • nix-review rev HEAD builds quite a lot of stuff, but never so far what was in the pull request
  • nix-review rev $git_head_commit_id same
  • nix-review rev HEAD --branch master same
  • nix-review rev HEAD --branch $previous_git_commit_id seems to work better, as in the two versions of nixpkgs that are evaluated to find the changed packages look correct now, but still builds something, but not what is in the pull request

At this point I'm running out of ideas, all the other options do not seem applicable.

@dwt
Copy link
Author

dwt commented Apr 23, 2025

Some workarounds I'm using: create a draft pull request and check it with nix-review pr $id, or just build with nix build .#$package and provide feedback that way.

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

No branches or pull requests

1 participant