-
Notifications
You must be signed in to change notification settings - Fork 28
[chore] Enable spell-check and fix spelling #295
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
[chore] Enable spell-check and fix spelling #295
Conversation
4c737cf
to
74f1c4c
Compare
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 doing all this @chalin!
Signed-off-by: Patrice Chalin <[email protected]>
74f1c4c
to
02e2c9e
Compare
analyses/0007-porter.md
Outdated
title: Porter tech docs assessment draft | ||
tags: porter | ||
title: Porter docs analysis |
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.
@thisisobate @nate-double-u: I'm assuming that this isn't a draft anymore, as suggested by the title
, so I updated the title.
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.
Technically it is still a draft as it's not been presented to the project yet.
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.
Oh! But the date is back in 2023! Do you want me to revert the title change?
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.
Waiting for your final approval @nate-double-u. If this LGTY, please go ahead and merge now that all checks are passing.
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.
Actually, let me clean up a few linter issues first.
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.
Done. PTAL
Signed-off-by: Patrice Chalin <[email protected]>
b588d50
to
6026501
Compare
Signed-off-by: Patrice Chalin <[email protected]>
- https://github.com/getporter/operator (https://getporter.org/operator site) | ||
|
||
# Porter Docs Assessment | ||
# Porter Docs Analysis |
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.
@nate-double-u : I actually dropped the front-matter title given that this in-page title takes precidence.
/cc @dwelsch-esi