[PR #18261] Upgrade to check-spelling v0.0.24 #31488

Open
opened 2026-01-31 09:47:35 +00:00 by claunia · 0 comments
Owner

Original Pull Request: https://github.com/microsoft/terminal/pull/18261

State: closed
Merged: Yes


Summary of the Pull Request

This upgrades to v0.0.24.

A number of GitHub APIs are being turned off shortly, so you need to upgrade or various uncertain outcomes will occur.

There are some minor bugs that I'm aware of and which I've fixed since this release (including a couple I discovered while preparing this PR).

References and Relevant Issues

Detailed Description of the Pull Request / Additional comments

There's a new accessibility forbidden pattern:

Do not use (click) here links
For more information, see:

(?i)(?:>|\[)(?:(?:click |)here|link|(?:read |)more)(?:</|\]\()

It's not enabled because I'm waiting for:

Validation Steps Performed

https://github.com/check-spelling-sandbox/terminal/actions/
https://github.com/check-spelling-sandbox/terminal/actions/runs/12076096237#summary-33676944561 is what you would have seen if this was running w/o the fixes included
https://github.com/check-spelling-sandbox/terminal/actions/runs/12032865816#summary-33545680702 is what I worked from

PR Checklist

  • Closes #xxx
  • Tests added/passed
  • Documentation updated
    • If checked, please file a pull request on our docs repo and link it here: #xxx
  • Schema updated (if necessary)
**Original Pull Request:** https://github.com/microsoft/terminal/pull/18261 **State:** closed **Merged:** Yes --- ## Summary of the Pull Request This upgrades to [v0.0.24](https://github.com/check-spelling/check-spelling/releases/tag/v0.0.24). A number of GitHub APIs are being turned off shortly, so you need to upgrade or various uncertain outcomes will occur. There are some minor bugs that I'm aware of and which I've fixed since this release (including a couple I discovered while preparing this PR). ## References and Relevant Issues ## Detailed Description of the Pull Request / Additional comments There's a new accessibility forbidden pattern: > Do not use `(click) here` links > For more information, see: > * https://www.w3.org/QA/Tips/noClickHere > * https://webaim.org/techniques/hypertext/link_text > * https://granicus.com/blog/why-click-here-links-are-bad/ > * https://heyoka.medium.com/dont-use-click-here-f32f445d1021 ```pl (?i)(?:>|\[)(?:(?:click |)here|link|(?:read |)more)(?:</|\]\() ``` It's not enabled because I'm waiting for: - [x] #18258 ## Validation Steps Performed https://github.com/check-spelling-sandbox/terminal/actions/ https://github.com/check-spelling-sandbox/terminal/actions/runs/12076096237#summary-33676944561 is what you would have seen if this was running w/o the fixes included https://github.com/check-spelling-sandbox/terminal/actions/runs/12032865816#summary-33545680702 is what I worked from ## PR Checklist - [ ] Closes #xxx - [ ] Tests added/passed - [ ] Documentation updated - If checked, please file a pull request on [our docs repo](https://github.com/MicrosoftDocs/terminal) and link it here: #xxx - [ ] Schema updated (if necessary)
claunia added the pull-request label 2026-01-31 09:47:35 +00:00
Sign in to join this conversation.
No Label pull-request
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#31488