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
idea: numerous fixes and improvements #24526
Conversation
JetBrains IDEs were put under wrong scope which was confusing. Fixes NixOS#24453
It is no longer present on JetBrains download servers and is very outdated.
@miltador, thanks for your PR! By analyzing the history of the files in this pull request, we identified @edwtjo, @chris-martin and @globin to be potential reviewers. |
I really like the changes. However I think that you should document the breaking change done by renaming |
@Ma27, good catch! Added a note to release notes. |
I like the rename. Please add an deprecated alias to |
@chris-martin, done. |
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.
I really like these changes, so I approve (did not test-build, though).
Motivation for this change
Improve JetBrains IDE packages.
Things done
(nix.useSandbox on NixOS,
or option
build-use-sandbox
innix.conf
on non-NixOS)
nix-shell -p nox --run "nox-review wip"
./result/bin/
)