next.js
Use and enforce exhaustive switch statements for work unit store
#81577

Merged

Use and enforce exhaustive switch statements for work unit store #81577

unstubbable merged 7 commits into canary from hl/work-unit-store-exhaustive-check

ijjk added created-by: Next.js team

ijjk added type: next

unstubbable marked this pull request as ready for review 1 year ago

unstubbable requested a review from

gnoff 1 year ago

unstubbable requested a review from

lubieowoce 1 year ago

Add eslint rules to enforce exhaustive `switch` for workUnitStore

fb5e4350

Simplify eslint rules

ba6dd1d3

Switch all `workUnitStore.type` checks to exhaustive switch statements

85f4749d

Fix a typo

bbe3bebe

Handle two missed switches

0da6e68e

Handle new case after rebasing

72857a92

unstubbable force pushed from f58f8d55 to 72857a92 1 year ago

gnoff approved these changes on 2025-07-14

Use top-level functions instead of closures

6b903148

unstubbable merged 1c1d9b5f into canary 1 year ago

unstubbable deleted the hl/work-unit-store-exhaustive-check branch 1 year ago

github-actions added locked

github-actions locked as resolved and limited conversation to collaborators 1 year ago

Reviewers

gnoff

lubieowoce

Assignees

No one assigned

Labels

type: next created-by: Next.js team locked

Milestone

No milestone