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

proofread #1211

wants to merge 1 commit into
base: master


None yet
3 participants

railsnewbie257 commented Jan 14, 2017

Main changes in


  • small spelling and grammar changes


  • date references "month," -> "month:" (use colon) to conform to date formate used later in the file.
  • numbers in tables are right justified.
  • contemplated putting ","(comma) in large numbers, was not sure of intent showing without
    (very informative and impressive history)


  • two small changes "ideally"->"ideal", "you"->"you'd"

This comment has been minimized.

mention-bot commented Jan 14, 2017

@railsnewbie257, thanks for your PR! By analyzing the history of the files in this pull request, we identified @bagder and @mundry to be potential reviewers.


bagder approved these changes Jan 14, 2017

@bagder bagder closed this in 02ee3b2 Jan 14, 2017


This comment has been minimized.


bagder commented Jan 14, 2017


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