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.

Sign up for GitHub

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

Jump to bottom

Add several spelling corrections #2396

Merged
merged 3 commits into from
Jul 13, 2022
Merged

Add several spelling corrections #2396

merged 3 commits into from
Jul 13, 2022

Conversation

kianmeng
Copy link
Contributor

@kianmeng kianmeng commented Jul 7, 2022

No description provided.

peternewman
peternewman requested changes Jul 12, 2022
View reviewed changes
Copy link
Collaborator

@peternewman peternewman left a comment

Choose a reason for hiding this comment

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

A few minor tweaks.

codespell_lib/data/dictionary.txt Outdated Show resolved Hide resolved
codespell_lib/data/dictionary.txt Show resolved Hide resolved
@luzpaz
Copy link
Collaborator

luzpaz commented Jul 12, 2022

Apologies, I though this was one of my personal PRs. I merged the changes @peternewman requested. :cringe:

@peternewman peternewman added the dictionary Changes to the dictionary label Jul 12, 2022
peternewman
peternewman approved these changes Jul 13, 2022
View reviewed changes
Copy link
Collaborator

@peternewman peternewman left a comment

Choose a reason for hiding this comment

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

LGTM, but it would be nice to check that @kianmeng is happy with @luzpaz unexpected changes.

@luzpaz
Copy link
Collaborator

luzpaz commented Jul 13, 2022

Yes, please weigh-in @kianmeng

@kianmeng
Copy link
Contributor Author

@peternewman @luzpaz I'm good. 😄

@peternewman peternewman merged commit 39bc9b2 into codespell-project:master Jul 13, 2022
@kianmeng
Copy link
Contributor Author

🥳 🥳 🥳 🥳 🥳

@luzpaz
Copy link
Collaborator

luzpaz commented Jul 14, 2022

@kianmeng again, apologies. Thanks for your contribution!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@peternewman peternewman peternewman approved these changes

Assignees
No one assigned
Labels
dictionary Changes to the dictionary
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@kianmeng @luzpaz @peternewman