nix
nix copied to clipboard
`ValidPathInfo` JSON format should use `null` not omit field
Motivation
Context
Blocked on #9994, tests will fail until this
A little progress on #10311
Priorities and Process
Add :+1: to pull requests you find important.
The Nix maintainer team uses a GitHub project board to schedule and track reviews.
Discussed during the Nix maintainers meeting on 2024-02-26. Idea approved, but :
- Should be mentioned in the release notes
- Should be reflected in the json guidelines policy
-
@edolstra: Potentially breaking change, although probably not a big one
- Also only happens in the new command-line
-
@thufschmitt: Do we care about having these
nullfields?- @roberth: Have a slightly different semantics, potentially interesting to distinguish
-
Idea approved
-
Should be mentioned in the release notes
-
Should be reflected in the json guidelines policy
This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/2024-02-28-nix-team-meeting-129/40499/1
This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/2024-06-03-nix-team-meeting-minutes-149/46582/1