-
-
Notifications
You must be signed in to change notification settings - Fork 16.8k
cue.validator: fix deprecated --strict flag #431813
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
The --strict flag was deprecated in CUE 0.14.0 and causes NixOS builds to fail with: '--strict is deprecated; use "jsonschema+strict:" as shown in "cue help filetypes"' Since strict validation is now the default behavior in CUE, we can safely remove this flag without changing functionality.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for fixing this! I've created #431906 to prevent similar issues from making it to nixos-unstable
in the future.
@aaronjheng for review (not sure why CI didn't ping here) |
Is something blocking this? |
yes, it's pending a committer noticing and clicking the merge button |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed my build, thanks!
@ElvishJerricco do you or another committer have a chance to review? |
The --strict flag was deprecated in CUE 0.14.0 and causes NixOS builds to fail with: '--strict is deprecated; use "jsonschema+strict:" as shown in "cue help filetypes"'
Since strict validation is now the default behavior in CUE, we can safely remove this flag without changing functionality.
Things done
passthru.tests
.nixpkgs-review
on this PR. See nixpkgs-review usage../result/bin/
.Add a 👍 reaction to pull requests you find important.