-
Notifications
You must be signed in to change notification settings - Fork 21.3k
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
Rewrite activerecord/RUNNING_UNIT_TESTS.rdoc
#10601
Conversation
@@ -1,3 +1,7 @@ | |||
* Rewrite RUNNING_UNIT_TESTS.rdoc | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No need CHANGELOG entry
Changelog reverted. |
Much improved, thank you! |
Could you squash your commits? |
If you don't have the environment set make sure to read | ||
|
||
http://edgeguides.rubyonrails.org/contributing_to_ruby_on_rails.html#testing-active-record | ||
If you don't have an environment in which to run tests, read |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this could be simpler like If you don't have an environment for running tests
maybe?
`RUNNING_UNIT_TESTS` is written in an unclear and confusing manner. This commit attempts to correct that. Thanks to zzak and vipulnsward for feedback.
Commits have been squashed, @rafaelfranca. Thanks for the feedback, @zzak and @vipulnsward. |
Thank you so much |
Rewrite `activerecord/RUNNING_UNIT_TESTS.rdoc`
I rewrote
activerecord/RUNNING_UNIT_TESTS.rdoc
to acheive, in my opinion, a greater level of clarity. The file has been reorganized, a few explanations were expanded, and one of the external hyperlinks was changed.