diff options
author | Bundlerbot <bot@bundler.io> | 2019-01-24 00:18:27 +0000 |
---|---|---|
committer | Bundlerbot <bot@bundler.io> | 2019-01-24 00:18:27 +0000 |
commit | e58436ef762ef1120024bbcda2620637f0484547 (patch) | |
tree | 9f378566d14c95dbea307692c3012018db90ed62 /bundler.gemspec | |
parent | e66e6f2fb7d595151bd77a342a07c9cd15e3cac0 (diff) | |
parent | 5c78fef54a595da1be1b395f4ab4533e345a8c04 (diff) | |
download | bundler-e58436ef762ef1120024bbcda2620637f0484547.tar.gz |
Merge #6926
6926: Move Rubocop development dependency into gemspec r=hsbt a=alyssais
### What was the end-user problem that led to this PR?
The problem was that tools like RubyGems.org or Bundix that use the gemspec as the source of truth for dependencies didn't pick up on the development dependency on Rubocop.
### What was your diagnosis of the problem?
My diagnosis was that the reason for this, which was that Rubocop had a minimum Ruby version of 2.0.0, was no longer an issue, since Bundler 2 doesn't support Rubies that old anyway.
### What is your fix for the problem, implemented in this PR?
My fix was to move the Rubocop requirement into the gemspec.
### Why did you choose this fix out of the possible options?
I chose this fix because it addresses the problem I was having, feels right, and the old workaround doesn't look like it's necessary any more.
Co-authored-by: Alyssa Ross <hi@alyssa.is>
Diffstat (limited to 'bundler.gemspec')
-rw-r--r-- | bundler.gemspec | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/bundler.gemspec b/bundler.gemspec index 8dda139c0a..5d494bcb0a 100644 --- a/bundler.gemspec +++ b/bundler.gemspec @@ -46,6 +46,7 @@ Gem::Specification.new do |s| s.add_development_dependency "rdiscount", "~> 2.2" s.add_development_dependency "ronn", "~> 0.7.3" s.add_development_dependency "rspec", "~> 3.6" + s.add_development_dependency "rubocop", "= 0.50.0" base_dir = File.dirname(__FILE__).gsub(%r{([^A-Za-z0-9_\-.,:\/@\n])}, "\\\\\\1") s.files = IO.popen("git -C #{base_dir} ls-files -z", &:read).split("\x0").select {|f| f.match(%r{^(lib|exe)/}) } |