Skip to content

ci: run only a single Windows check - #242

Merged
angeloashmore merged 2 commits into
mainfrom
aa/decrease-windows-ci
Jul 29, 2026
Merged

ci: run only a single Windows check#242
angeloashmore merged 2 commits into
mainfrom
aa/decrease-windows-ci

Conversation

@angeloashmore

@angeloashmore angeloashmore commented Jul 29, 2026

Copy link
Copy Markdown
Member

Resolves: https://prismic-team.slack.com/archives/C014VAACCQL/p1784933535521479

Description

This PR reduces the Windows test CI jobs from three (Node.js 20, 22, 24) to one (Node.js 24). As a result, we cut down on failures and wait time.

Reasoning: We only need to run Node 20, 22, 24 on some OS and one Windows sanity-check.

Checklist

  • If my changes require tests, I added them.
  • If my changes affect backward compatibility, it has been discussed.
  • If my changes require an update to the CONTRIBUTING.md guide, I updated it.

Preview

How to QA 1


Note

Low Risk
CI-only matrix change with no application or runtime behavior changes; slightly less Node version coverage on Windows only.

Overview
Narrows the test-win32 job in validate.yml so Windows unit tests run only on Node.js 24, instead of a matrix across 20, 22, and 24.

Linux test still exercises all three Node versions; Windows remains a single sanity check with less CI time and fewer flaky Windows-only failures.

Reviewed by Cursor Bugbot for commit be6f021. Bugbot is set up for automated code reviews on this repo. Configure here.

Footnotes

  1. Please use these labels when submitting a review:
    ❓ #ask: Ask a question.
    💡 #idea: Suggest an idea.
    ⚠️ #issue: Strongly suggest a change.
    🎉 #nice: Share a compliment.

@angeloashmore
angeloashmore marked this pull request as ready for review July 29, 2026 22:00
@angeloashmore
angeloashmore merged commit dd9f0e0 into main Jul 29, 2026
15 checks passed
@angeloashmore
angeloashmore deleted the aa/decrease-windows-ci branch July 29, 2026 22:48
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.

2 participants