Skip to content

Make nix path-info follow the JSON guidelines#14708

Merged
Ericson2314 merged 1 commit intoNixOS:masterfrom
obsidiansystems:version-path-info-outer
Dec 4, 2025
Merged

Make nix path-info follow the JSON guidelines#14708
Ericson2314 merged 1 commit intoNixOS:masterfrom
obsidiansystems:version-path-info-outer

Conversation

@Ericson2314
Copy link
Member

Motivation

The guidelines say for sake of extensibility, the top-level item must be a record, not a map. And indeed, we should put storeDir there, so the advice is well-headed.

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 4, 2025 04:43
@github-actions github-actions bot added documentation new-cli Relating to the "nix" command with-tests Issues related to testing. PRs with tests have some priority labels Dec 4, 2025
@edolstra edolstra changed the title Make nix path-info follow the JSON guildelines Make nix path-info follow the JSON guidelines Dec 4, 2025
@Ericson2314 Ericson2314 added this pull request to the merge queue Dec 4, 2025
Merged via the queue into NixOS:master with commit a4fc386 Dec 4, 2025
17 checks passed
@Ericson2314 Ericson2314 deleted the version-path-info-outer branch December 4, 2025 17:45
@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