-
Notifications
You must be signed in to change notification settings - Fork 62
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
Recommend sembr for docs #249
Conversation
Signed-off-by: Sajay Antony <sajaya@microsoft.com>
@TerryHowe - Do you think we should avoid line breaks at 80 and just do one line a sentence. I couldn't understand if sembr recommends the line breaks or one new sentence per line or both. |
I don't need line breaks at 80, but I could go either way on that. |
On second thought, I'd prefer we didn't mandate 80 chars. I'd be breaking that all the time. |
Agreed. I also find holding the 80 char limit hard :) |
Does this need to be a draft still? |
Update review status @TerryHowe |
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
@shizhMSFT @FeynmanZhou any concerns? |
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
Preview link - https://deploy-preview-249--oras-project.netlify.app/docs/community/contributing_guide#development-environment-setup Signed-off-by: Sajay Antony <sajaya@microsoft.com>
Preview link - https://deploy-preview-249--oras-project.netlify.app/docs/community/contributing_guide#development-environment-setup