-
Notifications
You must be signed in to change notification settings - Fork 691
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
Add deprecation warnings to Nix integration #9205
Conversation
c37ab91
to
9cf3a97
Compare
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.
Very good.
9cf3a97
to
8d97abe
Compare
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.
QA notes are missing, and there is a redundant WARNING. Otherwise looks good.
8d97abe
to
649e4fc
Compare
53ef471
to
623fd89
Compare
@yvan-sraka good job, now it's time to update the CLI output test suite and it should be good |
035c0e7
to
ea553b2
Compare
46a9361
to
4e8a3f1
Compare
I updated the PR, now the tests are green :) |
4e8a3f1
to
5581ff5
Compare
This is a follow-up of haskell#9205
🎉 |
This PR modifies
cabal
behavior by adding deprecation warnings to Nix integration.This has been discussed on the discourse.
Checklist:
Bonus points for added automated tests!
QA Notes:
cabal --help
should display the[DEPRECATED]
label next to each Nix integration-related flag in the list ;cabal some-v1-command
with the--enable-nix
or--nix=True
flag should display a warning about the deprecation of the feature and provide a link to the online documentation for further information.