Skip to content

Conversation

@ilikecodewithmyhand
Copy link
Contributor

Closes #3184

Reduce typos lint noise from 16,944 errors to 0 by excluding specs/ submodule and add typos-cli installation check with user prompt.

@codecov
Copy link

codecov bot commented Dec 11, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.7%. Comparing base (01de8f5) to head (75f4a48).
⚠️ Report is 9 commits behind head on main.
✅ All tests successful. No failed tests found.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@einar-oplabs
Copy link
Collaborator

This is great! What about the actual typos that are in the repo?

@ilikecodewithmyhand
Copy link
Contributor Author

This is great! What about the actual typos that are in the repo?

Good question! I verified this by running typos with and without the specs/ exclusion:

  • Without excluding specs/: 16,944 errors (all from the specs/ submodule)
  • Main kona codebase only: 0 errors

All typos originate from the specs/ submodule, which is maintained upstream. The main kona codebase has zero typos, so we're not hiding any real issues by excluding specs

@github-actions
Copy link
Contributor

github-actions bot commented Jan 7, 2026

This pull request has been automatically marked as stale because it has been inactive for 3 weeks.
It will be closed in 1 week if no further activity occurs.
If you believe this PR should remain open, please add the M-prevent-stale label or leave a comment.

@github-actions github-actions bot added M-stale and removed M-stale labels Jan 7, 2026
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.

chore(lint): make lint-typos helpful

2 participants