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
rubocop: init at 0.47.0 #21934
rubocop: init at 0.47.0 #21934
Conversation
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.
Just two observations. It's looking good overall.
|
||
bundlerEnv rec { | ||
name = "rubocop-${version}"; | ||
version = "0.47.0"; |
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.
Instead of this it's also possible to use pname = "rubocop";
. This will automatically extract the version from the gemset.nix
and hide any bin that are not part of the main "rubocop" package.
|
||
gemfile = ./Gemfile; | ||
lockfile = ./Gemfile.lock; | ||
gemset = ./gemset.nix; |
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.
There is a new option called gemdir = ./.;
that can be used to automatically infer all these files.
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.
@zimbatm how about throwing a deprecation message for the old variant?
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.
👍 thanks @zimbatm! Will make life easier for future packages I might need to build :) |
Use gemdir and pname as suggested by @zimbatm Fix silly typo
Motivation for this change
To add the rubocop linter as a global package (for support with text editors that don't know how to use bundler or docker or nix-shell)
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/
)