Skip to content

fix: Github diffs URL#1564

Merged
dasJ merged 1 commit intoNixOS:masterfrom
d-goldin:fix/github-diff-url
Jan 30, 2026
Merged

fix: Github diffs URL#1564
dasJ merged 1 commit intoNixOS:masterfrom
d-goldin:fix/github-diff-url

Conversation

@d-goldin
Copy link
Contributor

In #1549 diffs were offloaded to github for performance reasons.

While in some endpoints github accepts .git suffixed in the repository name, in the comparison endpoint this does not seem to be the case.

Specifically, on the main nixos org hydra this isn't working:

Example job: https://hydra.nixos.org/build/320178054

Generates a comparison link like so:
https://github.com/NixOS/nixpkgs.git/compare/078d69f03934859a181e81ba987c2bb033eebfc5...1cd347bf3355fce6c64ab37d3967b4a2cb4b878c

This just stips away the suffix and seems to work fine in local testing.

cc @dasJ

In NixOS#1549 diffs were
offloaded to github for performance reasons.

While in some endpoints github accepts `.git` suffixed in the
repository name, in the comparison endpoint this does not seem
to be the case.

Specifically, on the main nixos org hydra this isn't working:

Example job: https://hydra.nixos.org/build/320178054

Generates a comparison link like so:
https://github.com/NixOS/nixpkgs.git/compare/078d69f03934859a181e81ba987c2bb033eebfc5...1cd347bf3355fce6c64ab37d3967b4a2cb4b878c

This just stips away the suffix and seems to work fine in local
testing.
@dasJ dasJ added this pull request to the merge queue Jan 30, 2026
@dasJ dasJ added the www hydra-www component label Jan 30, 2026
Merged via the queue into NixOS:master with commit a9581bc Jan 30, 2026
2 checks passed
@d-goldin d-goldin deleted the fix/github-diff-url branch January 30, 2026 21:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

www hydra-www component

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants