Skip to content
This repository was archived by the owner on Apr 6, 2026. It is now read-only.

Update hf-nix and remove sanitiseHeaderPathsHook workaround#201

Merged
danieldk merged 2 commits into
mainfrom
bump-hf-nix
Aug 29, 2025
Merged

Update hf-nix and remove sanitiseHeaderPathsHook workaround#201
danieldk merged 2 commits into
mainfrom
bump-hf-nix

Conversation

@danieldk

@danieldk danieldk commented Aug 28, 2025

Copy link
Copy Markdown
Member

No description provided.

`sanitiseHeaderPathsHook` was removed upstream in favor of another
solution that does not cause rebuild in CUDA packages.
@danieldk danieldk marked this pull request as ready for review August 28, 2025 13:26

@drbh drbh left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! also I think the new formatter's format is nice (a bit more concise 👍)

@danieldk

Copy link
Copy Markdown
Member Author

also I think the new formatter's format is nice (a bit more concise 👍)

Yeah, indeed, it looks much closer to what one would do with manual formatting.

@danieldk danieldk merged commit a9a85ef into main Aug 29, 2025
9 of 10 checks passed
@danieldk danieldk deleted the bump-hf-nix branch August 29, 2025 15:43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants