Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Make the Bundler warning less scary and more friendly #275

Merged
merged 1 commit into from

2 participants

@mcmire

Fix for #274

@mcmire

It's true the message may be a bit too obvious now -- if you're not happy about it, let me know and I'll see what I can do to edit it down.

@thibaudgg thibaudgg merged commit d60e701 into guard:master
@thibaudgg
Owner

Oops it breaks the specs badly :( http://travis-ci.org/#!/guard/guard/builds/1072250

@mcmire

Whoops! Sorry. I should know better than to not run the tests. If you pull the branch again, I've fixed them.

@thibaudgg
Owner

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Apr 11, 2012
  1. @mcmire
This page is out of date. Refresh to see the latest.
Showing with 8 additions and 1 deletion.
  1. +8 −1 lib/guard/cli.rb
View
9 lib/guard/cli.rb
@@ -159,7 +159,14 @@ def show
#
def verify_bundler_presence
if File.exists?('Gemfile') && !ENV['BUNDLE_GEMFILE']
- ::Guard::UI.warning 'You are using Guard outside of Bundler, this is dangerous and may not work. Using `bundle exec guard` is safer.'
+ ::Guard::UI.info <<EOT
+
+Guard here! It looks like your project has a Gemfile, yet you are running
+`guard` outside of Bundler. If this is your intent, feel free to ignore this
+message. Otherwise, consider using `bundle exec guard` to ensure your
+dependencies are loaded correctly.
+(You can run `guard` with --no-bundler-warning to get rid of this message.)
+EOT
end
end
Something went wrong with that request. Please try again.