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
Copyedit docs #6275
Copyedit docs #6275
Conversation
also capitalize other words
Thanks for PR 😄, other than the two small nits, this looks great! |
Should there be a comma after each dependent clause, as in: "Before you enable HTTPS[,] make sure that you have valid SSL/TLS certificates."? There are a lot of sentences like this without the comma and a few with it. |
Please undo also changes in vendor directory |
Codecov Report
@@ Coverage Diff @@
## master #6275 +/- ##
==========================================
- Coverage 38.88% 38.87% -0.01%
==========================================
Files 359 359
Lines 51018 51018
==========================================
- Hits 19836 19834 -2
- Misses 28312 28314 +2
Partials 2870 2870
Continue to review full report at Codecov.
|
@aidan-fitz In terms of your question re: commas. I have no preference either way as long as it is consistent. |
Also fix comma splices Signed-off-by: Aidan Fitzgerald <acf67@cornell.edu>
Signed-off-by: Aidan Fitzgerald <acf67@cornell.edu>
There's still a few locale file changes in here. You should remove them. |
On the webhooks page, are these headings case-sensitive? X-Github-Delivery: f6266f16-1bf3-46a5-9ea4-602e06ead473
X-Github-Event: push |
If you Google them you'll see that GitHub actually intends them to be X-GitHub- so they should change too |
Signed-off-by: Aidan Fitzgerald <acf67@cornell.edu>
Signed-off-by: Aidan Fitzgerald <acf67@cornell.edu>
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.
One more nit. I have just been bitten by the gofmt change - so we change that.
Signed-off-by: Aidan Fitzgerald <acf67@cornell.edu>
@zeripath Is there anything in particular we should change? |
Add "the" Co-Authored-By: aidan-fitz <aidan-fitz@users.noreply.github.com>
Signed-off-by: Aidan Fitzgerald <acf67@cornell.edu>
I made a few copyedits to the documentation, including several large changes (e.g. Github → GitHub).