Add eqeqeq rule into the main rules block #525

Merged

Vylpes merged 1 commit from feature/517-eqeqeq-rule into release/0.12.0

2026-09-13 13:05:28 +01:00

Vylpes commented

2026-09-12 12:16:21 +01:00

Owner

#517

Vylpes added 1 commit

2026-09-12 12:16:21 +01:00

Add eqeqeq rule into the main rules block

Test / build (push) Successful in 2m12s

Details

7d267a8608

Vylpes

2026-09-12 12:16:26 +01:00

added the
type
change
label
requested review from Smithy-bot

Smithy-bot was assigned by Vylpes

2026-09-12 12:16:38 +01:00

Smithy-bot referenced this pull request

2026-09-13 12:13:09 +01:00

Finish repo-wide strict equality sweep and enable eqeqeq #517

Smithy-bot approved these changes

2026-09-13 12:13:09 +01:00

Smithy-bot left a comment

First-pass review (bot) — LGTM.

Checked out feature/517-eqeqeq-rule (7d267a8) and verified:

Moving eqeqeq into the main files-scoped rules block is correct now that those globs are src/**/* / tests/**/* (they match). The old standalone block was only needed when the globs were ./src / ./tests.
yarn lint is clean on the branch.
A deliberate == probe in src/bot.ts still fails with eqeqeq — rule remains enforced after the move.

No changes requested. Handing to human second review (VylpesTester, since Vylpes is the author).

**First-pass review (bot)** — LGTM. Checked out `feature/517-eqeqeq-rule` (`7d267a8`) and verified: - Moving `eqeqeq` into the main `files`-scoped rules block is correct now that those globs are `src/**/*` / `tests/**/*` (they match). The old standalone block was only needed when the globs were `./src` / `./tests`. - `yarn lint` is clean on the branch. - A deliberate `==` probe in `src/bot.ts` still fails with `eqeqeq` — rule remains enforced after the move. No changes requested. Handing to human second review (VylpesTester, since Vylpes is the author).

Smithy-bot requested review from VylpesTester

2026-09-13 12:13:09 +01:00