Skip to content

Fix typos and add GitHub Action job for /book #1121

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

Merged
merged 6 commits into from
Oct 27, 2023

Conversation

kianmeng
Copy link
Contributor

Found via codespell book/ -L crate

@tyranron tyranron added the k::documentation Related to project documentation label Nov 15, 2022
@tyranron tyranron added this to the 0.16.0 milestone Nov 15, 2022
Copy link
Member

@tyranron tyranron left a comment

Choose a reason for hiding this comment

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

@kianmeng could you add a codespell CI check? Would be nice to have one.

@kianmeng
Copy link
Contributor Author

@tyranron Should we fix all the typo errors in the whole repo before adding the CI job?

@tyranron
Copy link
Member

@kianmeng let's try and see. I'm not strictly opposed to this.

@kianmeng kianmeng force-pushed the fix-typos-for-book branch 2 times, most recently from 899acab to 48ff884 Compare December 4, 2022 07:23
@kianmeng
Copy link
Contributor Author

kianmeng commented Dec 4, 2022

@tyranron Sorry for the delay, I've added the GHA, but can't test and validate it from my end.

@kianmeng kianmeng requested a review from tyranron December 4, 2022 07:25
@kianmeng kianmeng changed the title Fix typos for book Fix typos and add GitHub Action job for /book Dec 4, 2022
Found via `codespell book/ -L crate`
@Snoupix
Copy link

Snoupix commented Aug 12, 2023

@tyranron (up)
Hey, are we far away from 0.16 release ?

@LegNeato
Copy link
Member

@tyranron Can this be merged? I don't see which changes are outstanding.

@tyranron tyranron added the enhancement Improvement of existing features or bugfix label Oct 27, 2023
Copy link
Member

@tyranron tyranron left a comment

Choose a reason for hiding this comment

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

@kianmeng thanks! And sorry for the delay!

@tyranron tyranron enabled auto-merge (squash) October 27, 2023 15:10
@tyranron tyranron merged commit 0fc95dd into graphql-rust:master Oct 27, 2023
@kianmeng
Copy link
Contributor Author

🥳 🥳 🥳 🥳 🥳

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Improvement of existing features or bugfix k::documentation Related to project documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants