-
Notifications
You must be signed in to change notification settings - Fork 183
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
Small readability updates #184
Conversation
Small readability updates.
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.
The wording/grammar changes look good to me.
I added a question in the one paragraph where you tried to simplify the text. There I am not sure how it was meant by the original author.
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.
@jpatrao I don't see any new edits or commits. Are you referring to your proposal in here? |
@spier Yes. |
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 again for the changes, this looks good to me right now :)
🎉 |
Small readability updates. Because a certain paragraph within the solution section was not clear to me ... and I guess to others too, I did rework it. Closer attention needs to be paid here because I did not intend to change the meaning of what was written.