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 typos #1835

Closed
wants to merge 3 commits into from
Closed

Fix typos #1835

wants to merge 3 commits into from

Conversation

szepeviktor
Copy link
Contributor

@szepeviktor szepeviktor commented Aug 31, 2024

Found a sea of misspellings.

@brillout Please consider using https://github.com/crate-ci/typos everywhere.

All the best to you!

@brillout
Copy link
Member

Thank you 💚

The CI is red, it seems that the source code changes aren't consistent?

Please consider using https://github.com/crate-ci/typos everywhere.

Up for a PR? You can fork the repository to test your CI changes.

@szepeviktor
Copy link
Contributor Author

Please consider using https://github.com/crate-ci/typos everywhere.

Up for a PR?

I let you add typos to CI: uses: crate-ci/typos@master

.typos.toml

[files]
extend-exclude = [
    ".git/",
    "CHANGELOG.md",
    "examples/i18n/locales/translations.js",
    "test-deprecated-design/i18n/locales/translations.js",
]
ignore-hidden = false

[default]
extend-ignore-re = [
    "\\[[0-9a-f]{7}\\]",
    "scss\\|styl\\|stylus",
]

@brillout
Copy link
Member

Thinking of it, it's best we only fix the typos for the documentation. The source code typo fixes would induce a lot of merge conflicts for the pending PRs and branches.

@szepeviktor
Copy link
Contributor Author

szepeviktor commented Aug 31, 2024

Now you have the tools. Please revert any directory you wish.

This PR is mainly a promotion for typos.

@brillout
Copy link
Member

Closing in favor of #1836. Thank you for getting this started!

@brillout brillout closed this Aug 31, 2024
@szepeviktor
Copy link
Contributor Author

Glad to contribute.

@brillout brillout mentioned this pull request Sep 1, 2024
@brillout
Copy link
Member

brillout commented Sep 1, 2024

Follow up: #1839.

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