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

Fix many typos #778

Merged
merged 5 commits into from
Dec 30, 2024
Merged

Fix many typos #778

merged 5 commits into from
Dec 30, 2024

Conversation

aphedges
Copy link
Contributor

This PR fixes a large amount of typos. Some of the fixes in the first commit are subjective, so please let me know if I need to revert them.

PyCharm found these mistakes.
I noticed this when using the CLI and changed it everywhere.
Copy link
Member

@acabal acabal left a comment

Choose a reason for hiding this comment

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

Thanks, only one minor issue that is not a typo

CHANGELOG.md Outdated
@@ -1256,15 +1256,15 @@

- Only emit m-071, m-072, m-041 if there are fewer than 2 sources

- Add Ukranian to the set of scripts that need a Latn suffix. Thanks to Robin Whittleton
- Add Ukrainian to the set of scripts that need a Latin suffix. Thanks to Robin Whittleton
Copy link
Member

Choose a reason for hiding this comment

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

This is incorrect, Latn is the actual language keyword used in these attributes

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This should be fixed by edf1c5e.

@acabal acabal merged commit d7a7e7f into standardebooks:master Dec 30, 2024
1 check passed
@acabal
Copy link
Member

acabal commented Dec 30, 2024

Thanks!

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