whitesmith / whitesmith/rubycritic
RubyCritic::RakeTask problem with bundler+rails+heroku
Nobody has claimed this yet.
- Dominant language
- Ruby
- Stars
- 3.5k
- Forks
- 234
- PR merge metrics
- No merged PRs in 30d
Description
When deploying a Rails app on Heroku that:
- uses the rubycritic rake task
- Gemfile declares rubycritic gem in the
development/testgroups
we get:
rake aborted!
LoadError: cannot load such file -- rubycritic/rake_task
This happens because of:
bundle install --without development:test
rake -P
I managed to work with
if defined?(Rails) && (Rails.env.development? || Rails.env.test?)
require 'rubycritic/rake_task'
RubyCritic::RakeTask.new do |task|
....
Not sure what to do here.
I do think that this gem does not make sense to be used in a production environment, so I'd like to place it in dev/test groups. I'd also expect for this to work "out of the box".
Do we just update the documentation to specify this use case?
Do we drop the rake task and instead supply a generator or simple example?
The rake task is extremely simple, just a RubyCritic::Cli::Application.new(options).
If we properly document options people should have no problems with it.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reproducing the failure with bundle install --without development:test followed by rake -P, then inspect the RubyCritic rake task and the documented CLI options. Resolve whether the intended outcome is documentation or a code change, and verify that a Rails app keeping RubyCritic in development/test does not fail during production task loading.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- ruby
- Domain
- tooling
- Issue type
- Bug
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100