Skip to content

stats.ts: fix error-able versions matching - #4293

Merged
Elchi3 merged 1 commit into
web-platform-dx:mainfrom
ddbeck:stats.ts-fix-error-able-versions-matching/wklmlrtwprwu
Sep 2, 2026
Merged

stats.ts: fix error-able versions matching#4293
Elchi3 merged 1 commit into
web-platform-dx:mainfrom
ddbeck:stats.ts-fix-error-able-versions-matching/wklmlrtwprwu

Conversation

@ddbeck

@ddbeck ddbeck commented Sep 2, 2026

Copy link
Copy Markdown
Collaborator

The previous iteration of this code matched versions like 2.34 but not 2.36. It should reject all versions before v3.35.

Prequel to #4294.

@ddbeck
ddbeck requested a review from Elchi3 September 2, 2026 15:41
@github-actions github-actions Bot added the tools and infrastructure Project internal tooling, such as linters, GitHub Actions, or repo settings label Sep 2, 2026

@Elchi3 Elchi3 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch! 👍

@Elchi3
Elchi3 merged commit e6f387c into web-platform-dx:main Sep 2, 2026
4 checks passed
@ddbeck
ddbeck deleted the stats.ts-fix-error-able-versions-matching/wklmlrtwprwu branch September 2, 2026 15:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tools and infrastructure Project internal tooling, such as linters, GitHub Actions, or repo settings

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants