diff options
| author | The Bundler Bot <bot@bundler.io> | 2018-03-05 00:18:55 +0000 |
|---|---|---|
| committer | The Bundler Bot <bot@bundler.io> | 2018-03-05 00:18:55 +0000 |
| commit | 1f6469ffa4aa92e3ab0fe0eea74ee461fcbfee4a (patch) | |
| tree | 0833d8c35951305a00908fa78a842b54e275beaf | |
| parent | 5fb9e9a08512828ad93663787ac9cffbda5d599e (diff) | |
| parent | ceef0184bcb34faeabaabc6dac57a5c40708d2ab (diff) | |
| download | bundler-1f6469ffa4aa92e3ab0fe0eea74ee461fcbfee4a.tar.gz | |
Auto merge of #6307 - alyssais:yanked_message, r=indirect
Improve yanked gem error message
The problem was that the error message for a yanked Gem told the user to:
> update your bundle to a different version of foo (1.0)
This is slightly misleading because in this example, foo 1.0 is the version that has been yanked. They should use any verion of foo **other** than 1.0.
My diagnosis was that the error message interpolated the gem specification (which converts to a string as "NAME (VERSION)") rather than just the name of the gem in this instance.
My fix was to change that occurrence of the gem specification to just the gem name.
I chose this fix because it was the only way that occurred to me to do it.
| -rw-r--r-- | lib/bundler/definition.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/bundler/definition.rb b/lib/bundler/definition.rb index 6b42fe0fb5..a50806ba4b 100644 --- a/lib/bundler/definition.rb +++ b/lib/bundler/definition.rb @@ -175,7 +175,7 @@ module Bundler raise GemNotFound, "Your bundle is locked to #{locked_gem}, but that version could not " \ "be found in any of the sources listed in your Gemfile. If you haven't changed sources, " \ "that means the author of #{locked_gem} has removed it. You'll need to update your bundle " \ - "to a different version of #{locked_gem} that hasn't been removed in order to install." + "to a version other than #{locked_gem} that hasn't been removed in order to install." end unless specs["bundler"].any? bundler = sources.metadata_source.specs.search(Gem::Dependency.new("bundler", VERSION)).last |