-
Notifications
You must be signed in to change notification settings - Fork 251
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
History.md #191
History.md #191
Conversation
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.
Some minor details that should be fixed before we merge this IMHO.
History.md
Outdated
* Feature/bulk pr for readme updates (Patrick Mulder #65, | ||
Troels Knak-Nielsen #74, Jeff Casimir #76) | ||
* Switches README format to markdown (Dennis Sivia #176) | ||
* Converts History.txt to markdown (Dennis Sivia #179) |
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.
Minor detail: I think the grammar here could be consistent with the previous lines (Port to RSpec 3, Allow better testing of cookies)
So these should be:
- Switch README format to Markdown
- Convert History.txt to markdown
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.
Thanks for the review.
Updated as 1 more commit.
After finishing review, we can squash it.
Updated
History.md
to latest master branch.You can see modified History here.
https://github.com/junaruga/rack-test/blob/feature/update-history/History.md