- Notifications
You must be signed in to change notification settings - Fork 47
Benefits of contributing article - style review and minor updates #201
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
Conversation
Update 06-conclusion-article.asciidoc
sync with remote
| Great! Will look next week. |
lenucksi left a comment
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 making this review, finding and fixing the broken parts. I've added a few extra typo fixes and a few comments one of which I think should definitely be addressed.
Co-Authored-By: Johannes Tigges <lenucksi@users.noreply.github.com>
Co-Authored-By: Johannes Tigges <lenucksi@users.noreply.github.com>
Co-Authored-By: Johannes Tigges <lenucksi@users.noreply.github.com>
rrrutledge left a comment
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.
LGTM. Improvements possible as well!
rrrutledge left a comment
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.
LGTM
lenucksi left a comment
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 changes, looks good to me now.
No description provided.