-
Notifications
You must be signed in to change notification settings - Fork 47
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
treat warnings in install.packages as errors #92
base: master
Are you sure you want to change the base?
Conversation
Now I see what you mean... Amending. |
The test now tests what it's supposed to test. Do we want warnings in |
bump Warnings in |
I like adding it behind a flag -- how about a new one? (i imagine "my dependencies are having issues" being a separate worry from "i have warnings in my package".) |
Conflicts: scripts/travis-tool.sh
…STALL_ERRORS is nonempty
Updated to current |
warnings in
install_deps
are tolerated.This won't fix no #74, but still could be useful.