Skip to content

Fix spelling flagged by codespell - #6722

Merged
rdimitrov merged 1 commit into
mainfrom
fix/browser-binding-spelling
Sep 26, 2026
Merged

rdimitrov merged 1 commit into
mainfrom
fix/browser-binding-spelling

Conversation

@rdimitrov

Copy link
Copy Markdown
Member

Summary

  • The Spellcheck workflow runs only on pull requests, and the private advisory fork that delivered the browser-binding fix (merge commit 024c042) had no CI, so a comment spelled "unparseable" reached main unchecked. Codespell now fails on every PR branch cut from main, including the v0.51.3 release PR Release v0.51.3 #6721.
  • Changes the one word in a doc comment in pkg/authserver/server/handlers/browser_binding.go to "unparsable". No code change.

Type of change

  • Bug fix
  • New feature
  • Refactoring (no behavior change)
  • Dependency update
  • Documentation
  • Other (describe):

Test plan

  • Manual testing: codespell pkg/authserver/server/handlers/ reports nothing after the change.

Does this introduce a user-facing change?

No.

🤖 Generated with Claude Code

The Spellcheck workflow runs only on pull requests, and the advisory fork
that delivered the browser-binding fix had no CI, so this comment reached
main unchecked and now fails the check on the v0.51.3 release PR.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@github-actions github-actions Bot added the size/XS Extra small PR: < 100 lines changed label Sep 26, 2026
@rdimitrov
rdimitrov merged commit 0f5e721 into main Sep 26, 2026
21 of 22 checks passed
@rdimitrov
rdimitrov deleted the fix/browser-binding-spelling branch September 26, 2026 20:31
@rdimitrov rdimitrov mentioned this pull request Sep 26, 2026
2 tasks
@codecov

codecov Bot commented Sep 26, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.26%. Comparing base (024c042) to head (865c382).
⚠️ Report is 4 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #6722   +/-   ##
=======================================
  Coverage   79.26%   79.26%           
=======================================
  Files         801      801           
  Lines       81168    81168           
=======================================
+ Hits        64335    64339    +4     
+ Misses      16828    16824    -4     
  Partials        5        5           

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@github-actions github-actions Bot mentioned this pull request Sep 26, 2026
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/XS Extra small PR: < 100 lines changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant