Skip to content
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

Spellcheck fixes #749

Merged
merged 8 commits into from
Nov 15, 2024
Merged

Spellcheck fixes #749

merged 8 commits into from
Nov 15, 2024

Conversation

nhennigan
Copy link
Contributor

Fixed typos across the docs and created more extensive custom_wordlist

Corrected typos in docs and created more extensive custom_wordslist
Small typos in 2 files
Moonray docs now passing
tools directory docs now passing
added words to custom_wordlist to reflect updated repo. Also changed index files to remove errors
update working directory for spell check to docs/canonicalk8s
change spellcheck to manual launch rather than on every pull request. We want to update to use vale in the future so disabling for now
@nhennigan
Copy link
Contributor Author

Removed spellcheck from automatically running on every PR. Will update with vale implementation later. I think we can still merge this once approved to stop the ever failing test and fix existing typos in the docs.

@nhennigan nhennigan marked this pull request as ready for review November 15, 2024 18:01
@nhennigan nhennigan requested a review from a team as a code owner November 15, 2024 18:01
Copy link
Contributor

@eaudetcobello eaudetcobello left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work! Thanks

@nhennigan nhennigan merged commit 79f3145 into canonical:main Nov 15, 2024
15 of 17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants