Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Warn about excluding a previously excluded group from a local config file #1583

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
3 participants

This was the second time I was tripped by the behavior "If you run bundle install later, without any flag, bundler will remember that you last called bundle install --without production, and use that flag again."

For the principle of least surprise, I added a warning that will be shown when the without option is active, because of a local configuration file and not because of a command line parameter.

Test and code in the commit (I ran the rspec tests on 1.8.7 with a fairly old set of gems; I did not test it in travis).

This pull request fails (merged 5391928 into 6d431b8).

Owner

indirect commented Aug 4, 2013

Thanks for contributing, and sorry this patch never made it in... I was ambivalent about printing a warning inside settings. Master has code that prints out remembered settings at the end of every install, which should fix this problem. :)

@indirect indirect closed this Aug 4, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment