Skip to content

test: throwaway Node 24 required-check verification (WAL-1475) - #9613

Closed
gokulhost wants to merge 1 commit into
masterfrom
WAL-1475-verify-node24-required
Closed

test: throwaway Node 24 required-check verification (WAL-1475)#9613
gokulhost wants to merge 1 commit into
masterfrom
WAL-1475-verify-node24-required

Conversation

@gokulhost

Copy link
Copy Markdown
Contributor

Purpose

Throwaway verification PR for WAL-1475. Adds a test that fails only on Node 24 (modules/logger/test/unit/node24Verify.ts) to confirm whether a Node-24-only CI failure actually blocks merge under current branch protection.

Not intended to merge — will be closed once the checks list is observed.

What to check

  • Does unit-test (24.x) go red?
  • Does the all-checks umbrella job (which is a required check) also go red as a result?
  • Does GitHub actually block the merge button?

Ticket: WAL-1475

Verification-only PR for WAL-1475: confirms whether a test that fails
exclusively on Node 24 actually blocks merge under current branch
protection. To be closed without merging once verified.

Ticket: WAL-1475
@gokulhost
gokulhost requested review from a team as code owners August 31, 2026 19:36
@linear-code

linear-code Bot commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

WAL-1475

@gokulhost

Copy link
Copy Markdown
Contributor Author

Verification complete — see WAL-1475. unit-test (24.x) failed as expected, all-checks (the required check) cascaded to fail, and mergeStateStatus is BLOCKED. Node 24 failures already block merge today via the all-checks umbrella job. Closing without merging.

@gokulhost gokulhost closed this Aug 31, 2026
@gokulhost
gokulhost deleted the WAL-1475-verify-node24-required branch August 31, 2026 20:11
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