summaryrefslogtreecommitdiff
path: root/lib
diff options
context:
space:
mode:
authorBundlerbot <bot@bundler.io>2019-08-17 23:30:49 +0000
committerBundlerbot <bot@bundler.io>2019-08-17 23:30:49 +0000
commit64271ffdc2fb06c46d170991151f92687c4857a6 (patch)
tree00833b13a53b5e864716a055de4047fa94fb3ae8 /lib
parente16c72fcfdf1c72bbe0c36e7688e2ca297746849 (diff)
parentd18a83109e9c420aec2da15db85af97635eca232 (diff)
downloadbundler-64271ffdc2fb06c46d170991151f92687c4857a6.tar.gz
Merge #7302
7302: Bundler displays a duplicate gem entries warning even if gems only appear once per group r=hsbt a=davidstosik ### What was the end-user problem that led to this PR? I have a Gemfile where some gems appear in two distinct groups. When I run Bundle, the following warning is displayed: > Your Gemfile lists the gem #{gem} more than once. > You should probably keep only one of them. > Remove any duplicate entries and specify the gem **only once (per group)**. > While it's not a problem now, it could cause errors if you change the version of one of them later. Example Gemfile: ```rb source "https://rubygems.org" group :production do end group :development do gem "rake" end group :ci, optional: true do gem "rake" end ``` ### What was your diagnosis of the problem? I think the message is misleading, because it shows even if all gems are specified **only once per group**. ### What is your fix for the problem, implemented in this PR? I removed the _(per group)_ mention from the warning message in order to prevent confusion. ### Why did you choose this fix out of the possible options? I chose this fix because it was the simplest way to bring your attention to the problem. If Bundler's real intent is to allow the same gem to appear in multiple groups, as long as it does not appear more than once per group, then my suggestion is wrong, and instead, the logic around the warning message needs to be fixed. Co-authored-by: David Stosik <david.stosik+git-noreply@gmail.com>
Diffstat (limited to 'lib')
-rw-r--r--lib/bundler/dsl.rb2
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/bundler/dsl.rb b/lib/bundler/dsl.rb
index 534c9b5537..cc23f9b389 100644
--- a/lib/bundler/dsl.rb
+++ b/lib/bundler/dsl.rb
@@ -128,7 +128,7 @@ module Bundler
else
Bundler.ui.warn "Your Gemfile lists the gem #{current.name} (#{current.requirement}) more than once.\n" \
"You should probably keep only one of them.\n" \
- "Remove any duplicate entries and specify the gem only once (per group).\n" \
+ "Remove any duplicate entries and specify the gem only once.\n" \
"While it's not a problem now, it could cause errors if you change the version of one of them later."
end