summaryrefslogtreecommitdiff
path: root/exe
diff options
context:
space:
mode:
authorBundlerbot <bot@bundler.io>2018-11-17 15:25:58 +0000
committerBundlerbot <bot@bundler.io>2018-11-17 15:25:58 +0000
commitf160d15fe4e1cb25fad6a34ed9abd10b21f587f8 (patch)
treee588d45c9ae0d0a0879fe8508b855af9ecc9dd6b /exe
parent9cfe9bd9be4b44bde1fd71e6faab494509ceb7b6 (diff)
parent67320924db1f1910b17fae0a9268b02077874d26 (diff)
downloadbundler-f160d15fe4e1cb25fad6a34ed9abd10b21f587f8.tar.gz
Merge #6790
6790: Fix multiple source warning messages from `error` to `warn` r=greysteil a=jlw ### What was the end-user problem that led to this PR? I've been working on a few repos for most of this year that use a privately-hosted gem with the same name (but completely different purpose) as a gem hosted at rubygems.org - whenever I start with a clean copy and `bundle install` for the first time, I am presented with a warning in red that the gem was found in multiple sources. The red text makes me think that something went wrong, even though bundler did exactly what I wanted it to do. The user experience is wrong - a warning should not be red. ### What was your diagnosis of the problem? When I looked at the source, I saw immediately that the warning was using `Bundler.ui.error` instead of `Bundler.ui.warn`. This is a bug fix with a limited lifespan - after setting up a local copy of bundler I see that 2.0's handling of multiple sources A) is smarter and bypasses the error in most of the repos where I saw it with bundler 1.x; and B) fails early with a different error message in the remaining repos. ### What is your fix for the problem, implemented in this PR? The fix is simply switching to `Bundler.ui.warn` in this case. ### Why did you choose this fix out of the possible options? _I didn't see any other possible options._ Co-authored-by: Jeremy Weathers <jeremy@codekindly.com>
Diffstat (limited to 'exe')
0 files changed, 0 insertions, 0 deletions