Skip to content

test: match the missing-parameter error case-insensitively - #142

Merged
prrao87 merged 1 commit into
mainfrom
test/typos-case-insensitive-match
Nov 7, 2026
Merged

prrao87 merged 1 commit into
mainfrom
test/typos-case-insensitive-match

Conversation

@prrao87

@prrao87 prrao87 commented Nov 7, 2026

Copy link
Copy Markdown
Member

Why

The GitHub Actions bump in #141 moves Spell Check from typos 1.26.0 to 1.51.1, and the newer dictionary fails on one line: it reads the character class in a test's match="[Pp]arameter" as the misspelling arameter. Fixing it on main lets Dependabot rebase #141 onto a passing base without hand edits to its branch.

What changed

  • The test that checks a cached query still rejects a missing parameter now matches the error with (?i)parameter instead of [Pp]arameter. It accepts the same messages.

Validation

  • typos 1.51.1 and 1.26.0 both report nothing on the whole repository.
  • The plan cache tests pass (16 passed) and make lint passes locally. CI is pending.

🤖 Generated with Claude Code

typos 1.51.1, which the GitHub Actions bump in #141 moves Spell Check to,
reads the character class in `[Pp]arameter` as the misspelling
`arameter`. `(?i)parameter` matches the same messages.
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Nov 7, 2026 •

Copy link
Copy Markdown

Deploying lance-csr with  Cloudflare Pages  Cloudflare Pages

Latest commit: 595a430
Status: ✅  Deploy successful!
Preview URL: https://32cda2ea.lance-csr.pages.dev
Branch Preview URL: https://test-typos-case-insensitive.lance-csr.pages.dev

View logs

@prrao87
prrao87 marked this pull request as ready for review November 7, 2026 19:29
@prrao87
prrao87 merged commit 65c5cad into main Nov 7, 2026
4 checks passed
@prrao87
prrao87 deleted the test/typos-case-insensitive-match branch November 7, 2026 19:31
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.

1 participant