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

Minor fixes on documentation pages #1719

Merged
merged 3 commits into from Sep 7, 2021
Merged

Minor fixes on documentation pages #1719

merged 3 commits into from Sep 7, 2021

Conversation

pabluk
Copy link
Contributor

@pabluk pabluk commented Sep 7, 2021

This PR just fix some minor typos/spelling errors I found when I was reading the documentation site.

At first, I just wanted to try the Github Actions env vars, but after doing a copy/paste on my terminal I found out that there was a typo on the GITHUB_WORKFLOW variable name, so to avoid just creating a pull request with only a new byte on it I also run aspell check on all the *.rst files under docs/ and I checked all the URLs.

I didn't find any open issues related to this but I can open a new issue if you want.

Please feel free to request any changes or drop this PR if it's not relevant. Thanks

@ssbarnea ssbarnea added the bug label Sep 7, 2021
@ssbarnea
Copy link
Member

ssbarnea commented Sep 7, 2021

@pabluk Thanks! For obvious fixes like this we even prefer to just see a PR and bypass the issue creation.

In fact our project is using codespell inside .pre-commit-config.yaml which is catching some typing errors. I am not sure if aspell has support for pre-commit. If you find one way to add it to our pre-commit config, it could be useful in the future.

@ssbarnea ssbarnea enabled auto-merge (squash) September 7, 2021 11:12
@ssbarnea ssbarnea merged commit daea559 into ansible:main Sep 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants