Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Add word that failed in CI's spell check to the whitelist #349

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
koic merged 1 commit into rubocop:master from masato-bkn:add-word-to-codespell
Oct 19, 2023

Conversation

@masato-bkn
Copy link
Contributor

@masato-bkn masato-bkn commented Oct 19, 2023

When I opened #348, the CI's spellcheck failed.
https://github.com/rubocop/rails-style-guide/actions/runs/6570237096/job/17847343002?pr=348

I addressed the issue by adding the relevant word to the whitelist.

@masato-bkn masato-bkn marked this pull request as ready for review October 19, 2023 08:14
Copy link
Contributor Author

Sorry, I may have made a mistake in how I fix, so I'll go back to draft.

@masato-bkn masato-bkn changed the title (削除) add Mange to codespell.txt (削除ここまで) (追記) Add word that failed in CI's spell check to the whitelist. (追記ここまで) Oct 19, 2023
@masato-bkn masato-bkn changed the title (削除) Add word that failed in CI's spell check to the whitelist. (削除ここまで) (追記) Add word that failed in CI's spell check to the whitelist (追記ここまで) Oct 19, 2023
Copy link
Member

koic commented Oct 19, 2023

@masato-bkn Can you squash your commits into one?

Co-authored-by: Koichi ITO <koic.ito@gmail.com>
@koic koic merged commit ee8c5a1 into rubocop:master Oct 19, 2023
@masato-bkn masato-bkn deleted the add-word-to-codespell branch October 19, 2023 08:51
Copy link
Contributor

andyw8 commented Oct 19, 2023

🤔 shouldn't the spellchecker be configured to ignore URLs?

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

Reviewers

@koic koic koic left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /