-
Notifications
You must be signed in to change notification settings - Fork 0
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
Refactor README #9
Conversation
- move Acknowledgements into this file - align overall structure with our other open source projects like rubocop-cargosense - refactor text for consistency, clarity, and correctness Note that I don't think I meaningfully changed the meaning or intent behind anything and certainly didn't change the examples.
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.
Thank you so much! This looks great :)
My comments are mostly nits because the refactor read quite solidly to me. There's one substantive piece of feedback about the placement of the "Usage" section which I feel somewhat strongly about (though not overly -- I could be convinced). Happy to discuss.
Co-authored-by: Billy Lanchantin <[email protected]>
Co-authored-by: Billy Lanchantin <[email protected]>
Co-authored-by: Billy Lanchantin <[email protected]>
@billylanchantin All feedback has been addressed and I requested a re-review at your convenience. |
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.
Sorry, found 2 more nits that I missed!
Co-authored-by: Billy Lanchantin <[email protected]>
Co-authored-by: Billy Lanchantin <[email protected]>
Additional feedback implemented and review re-requested! |
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.
Awesome work! 🚢 it!
This PR updates the README with an eye toward clarity, brevity, and consistency.
Notable changes include:
Note that I didn't meaningfully changed the meaning or intent behind anything and certainly didn't change the examples.