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

ci: add spell checking with cspell #187

Merged
merged 2 commits into from
Aug 18, 2024

Conversation

adamhamlin
Copy link
Contributor

@adamhamlin adamhamlin commented Aug 18, 2024

Tip

The owner of this PR can publish a preview release by commenting /publish in this PR. Afterwards, anyone can try it out by running pnpm add radashi@pr<PR_NUMBER>.

Summary

Adds cspell to lint process to enforce spelling in virtually all files. Fixed any outstanding spelling mistakes/alarms.

Related issue, if any:

See https://github.com/orgs/radashi-org/discussions/171 for more info/discussion.

For any code change,

N/A

  • Related documentation has been updated, if needed
  • Related tests have been added or updated, if needed
  • Related benchmarks have been added or updated, if needed

Does this PR introduce a breaking change?

No

Bundle impact

Status File Size 1 Difference (%)
M src/async/defer.ts 415 +0 (+0%)
M src/curry/partob.ts 62 +0 (+0%)
M src/object/get.ts 190 +0 (+0%)
M src/string/camel.ts 357 +0 (+0%)
M src/string/capitalize.ts 152 +0 (+0%)
M src/string/dash.ts 334 +0 (+0%)
M src/string/similarity.ts 455 +0 (+0%)
M src/string/snake.ts 441 +0 (+0%)

Footnotes

  1. Function size includes the import dependencies of the function.

@aleclarson
Copy link
Member

This is awesome!

README.md Show resolved Hide resolved
src/object/get.ts Outdated Show resolved Hide resolved
@adamhamlin adamhamlin marked this pull request as ready for review August 18, 2024 18:36
@adamhamlin
Copy link
Contributor Author

@aleclarson I'm assuming the issues with the failing checks are unrelated to this PR

@radashi-org radashi-org deleted a comment from radashi-bot Aug 18, 2024
@aleclarson aleclarson merged commit 61e2750 into radashi-org:main Aug 18, 2024
8 checks passed
@adamhamlin adamhamlin deleted the add-spellchecking branch August 18, 2024 20:30
@aleclarson
Copy link
Member

Btw, great job on this, @adamhamlin 👏 You nailed it

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