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
docs: fix docs following #17526 #17570
Conversation
Documentation/spelling_wordlist.txt
Outdated
@@ -618,6 +620,7 @@ nat | |||
natMax | |||
natively | |||
neighMax | |||
neighbour |
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.
We have more occurrences of neighbor
(US spelling) overall in the repo. Would you mind updating the description of the related option instead, to be more consistent and to avoid adding the British spelling to this list, please?
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!
It seems we forgot to update `helm-values.rst` in cilium#17526, yielding issues when building documentation locally with `make render-docs`: ``` Please fix the following spelling mistakes: * Documentation/helm-reference.rst:876: (NeighDiscovery) * Documentation/helm-reference.rst:876: (arping) * Documentation/helm-reference.rst:877: (arping) * Documentation/helm-reference.rst:880: (NeighDiscovery) * Documentation/helm-reference.rst:881: (neighbour) * Documentation/helm-values.rst:876: (NeighDiscovery) * Documentation/helm-values.rst:876: (arping) * Documentation/helm-values.rst:877: (arping) * Documentation/helm-values.rst:880: (NeighDiscovery) * Documentation/helm-values.rst:881: (neighbour) If the words are not misspelled, run: Documentation/update-spelling_wordlist.sh NeighDiscovery arping neighbour ``` Since we already have more occurrences of `neighbor` (US spelling) in the repo, we replace `neighbour` with `neighbour` for consistency and then fix `helm-values.rst` + spelling wordlist. Signed-off-by: Nicolas Busseneau <nicolas@isovalent.com>
18aa0b1
to
4b9b9a1
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.
Thank you!
This does not need to run any CI (docs only changes), and relevant reviews are in (other automated reviews are due to a typo fix, can be ignored). Marking as ready to merge. |
It seems we forgot to update
helm-values.rst
in #17526, yielding issues when building documentation locally withmake render-docs
:Since we already have more occurrences of
neighbor
(US spelling) in the repo, we replaceneighbour
withneighbor
for consistency and then fixhelm-values.rst
+ spelling wordlist.