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

Converts History.txt to markdown #179

Merged

Conversation

dennissivia
Copy link

After discussing and agreeing on a convertion to markdown
(see github issue #168 ) this change renames the file and
cleans up the formatting.

NOTE: the content stays untouched.

After discussing and agreeing on a convertion to markdown
(see github issue rack#168 ) this change renames the file and
cleans up the formatting.
NOTE: the content stays untouched.
Copy link
Contributor

@perlun perlun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A very minor suggestion, but please fix and I'll approve afterwards.

History.md Outdated
Because #last_response is now a MockResponse instead of a Rack::Response,
#last_response.body now returns a string instead of an array.
Because #last_response is now a MockResponse instead of a Rack::Response, #last_response.body
now returns a string instead of an array.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Here we could also use backticks.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

jepp, i did it first but wass't sure if it would have enough value. I will update the PR accordingly. Thanks for the feedback.

According to the suggestions in the PR, more code snippets
haven been marked as code for proper rendering.
@dennissivia
Copy link
Author

@perlun PTAL

Copy link
Contributor

@perlun perlun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Please merge this (preferably using squash merge so we don't get more commits on master than necessary).

@dennissivia dennissivia merged commit bc60286 into rack:master May 19, 2017
@dennissivia dennissivia deleted the feature/convert-history-to-markdown branch May 19, 2017 19:00
alex-damian-negru pushed a commit to alex-damian-negru/rack-test that referenced this pull request Apr 5, 2021
* Converts History.txt to markdown

After discussing and agreeing on a convertion to markdown
(see github issue rack#168 ) this change renames the file and
cleans up the formatting.
NOTE: the content stays untouched.

* Improves code rendering

According to the suggestions in the PR, more code snippets
haven been marked as code for proper rendering.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants