Skip to content
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

add warnings option to test runner #27784

Merged

Conversation

@y-yagi
Copy link
Member

y-yagi commented Jan 24, 2017

Summary

Ruby's warning show useful information. However, there is no easy way to enable it with the current test runner(As far as I know). Therefore, I would like to be able to enable it by option. How about that?

@rails-bot
Copy link

rails-bot commented Jan 24, 2017

r? @sgrif

(@rails-bot has picked a reviewer for you, use r? to override)

@maclover7 maclover7 added the railties label Jan 24, 2017
@y-yagi
Copy link
Member Author

y-yagi commented Jan 29, 2017

r? @kaspth

@rails-bot rails-bot assigned kaspth and unassigned sgrif Jan 29, 2017
@rafaelfranca rafaelfranca merged commit 341fab8 into rails:master Jan 30, 2017
2 checks passed
2 checks passed
codeclimate no new or fixed issues
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@y-yagi y-yagi deleted the y-yagi:add_warnings_option_to_test_runner branch Jan 30, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

6 participants
You can’t perform that action at this time.