Skip to content

cli: error instead of panic - #411

Draft
stepbrobd wants to merge 4 commits into
mainfrom
panics
Draft

stepbrobd wants to merge 4 commits into
mainfrom
panics

Conversation

@stepbrobd

@stepbrobd stepbrobd commented Oct 2, 2026 •

Copy link
Copy Markdown
Member

return errors instead of panicking:

  • outside a hive, every command prints Could not find hive.nix or flake.nix in <dir> or any parent directory and exits 2
  • nix-info no longer loads the hive (i mean it doesnt need to right?)
  • apply-local returns evaluation errors (still panicked on reproducer of Panic when building a nix file with an unexpected type #26)
  • eval FILE reports a missing or non UTF-8 path

closes #26
closes #218 (same fix for the missing hive)

@stepbrobd
stepbrobd added this pull request to stack #412 October 2, 2026 10:14
@stepbrobd stepbrobd changed the title panics cli: error instead of panic Oct 2, 2026

This branch has not been deployed

No deployments
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

Successfully merging this pull request may close these issues.

Panic when building a nix file with an unexpected type

1 participant