forked from usesend/useSend
-
Notifications
You must be signed in to change notification settings - Fork 0
debt: restore the repository-wide lint gate #9
Copy link
Copy link
Open
Labels
area/packagespackages/: SDKs, editor, UI, and shared configurationpackages/: SDKs, editor, UI, and shared configurationarea/webapps/web: dashboard, API, delivery, campaigns, contactsapps/web: dashboard, API, delivery, campaigns, contactspriority/p2Someday; batched or opportunisticSomeday; batched or opportunistictype/debtCleanup or refactoring with no user-facing changeCleanup or refactoring with no user-facing change
Description
Activity
Metadata
Metadata
Assignees
Labels
area/packagespackages/: SDKs, editor, UI, and shared configurationpackages/: SDKs, editor, UI, and shared configurationarea/webapps/web: dashboard, API, delivery, campaigns, contactsapps/web: dashboard, API, delivery, campaigns, contactspriority/p2Someday; batched or opportunisticSomeday; batched or opportunistictype/debtCleanup or refactoring with no user-facing changeCleanup or refactoring with no user-facing change
Spec
Outcome.
pnpm lintis a reliable repository gate rather than a command that fails before it can evaluate a change.Scope. Restore ESLint resolution in the SDK workspace, triage the existing web warnings without weakening rules, and add lint to the always-running production PR gate once the baseline is green.
Done when.
pnpm lintpasses from a frozen-lockfile install on currentgrowthpath-mail, and the required GitHub Actions gate runs it without ignores or warning-budget exceptions.Evidence. During #6, top-level lint failed because
packages/sdkcould not resolveeslint;pnpm --filter=web lintthen reported 168 existing warnings under--max-warnings 0. Keeping that broken command in the production gate would make every PR unmergeable.