Skip to content

Improve wrong format message with nix hash convert#14720

Merged
edolstra merged 1 commit intoNixOS:masterfrom
obsidiansystems:nix-hash-convert-improve-error
Dec 8, 2025
Merged

Improve wrong format message with nix hash convert#14720
edolstra merged 1 commit intoNixOS:masterfrom
obsidiansystems:nix-hash-convert-improve-error

Conversation

@Ericson2314
Copy link
Member

Motivation

We have the machinery to make a more informative error, telling the user what format was actually encountered, and not just that it is not the format that was requested.

Context


Add 👍 to pull requests you find important.

The Nix maintainer team uses a GitHub project board to schedule and track reviews.

@Ericson2314 Ericson2314 requested a review from edolstra as a code owner December 5, 2025 16:25
@github-actions github-actions bot added documentation new-cli Relating to the "nix" command labels Dec 5, 2025
@Ericson2314 Ericson2314 force-pushed the nix-hash-convert-improve-error branch from 379c430 to 9c8a978 Compare December 5, 2025 16:25
We have the machinery to make a more informative error, telling the
user what format was actually encountered, and not just that it is not
the format that was requested.
@Ericson2314 Ericson2314 force-pushed the nix-hash-convert-improve-error branch from 9c8a978 to b618857 Compare December 5, 2025 20:12
@github-actions github-actions bot added the with-tests Issues related to testing. PRs with tests have some priority label Dec 5, 2025
@edolstra edolstra added this pull request to the merge queue Dec 8, 2025
Merged via the queue into NixOS:master with commit 32bc0ac Dec 8, 2025
16 checks passed
@Ericson2314 Ericson2314 deleted the nix-hash-convert-improve-error branch December 8, 2025 16:53
@edolstra edolstra mentioned this pull request Dec 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation new-cli Relating to the "nix" command with-tests Issues related to testing. PRs with tests have some priority

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants