Add changed-file lint gate#3257
Merged
nkaradzhov merged 2 commits intoredis:masterfrom Apr 24, 2026
Merged
Conversation
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Reviewed by Cursor Bugbot for commit 3968d41. Configure here.
bobymicroby
approved these changes
Apr 24, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

Summary
Motivation
This gates lint quality for files changed by new PRs without requiring the existing monorepo lint debt to be fixed in one large pass.
Validation
Note
Medium Risk
Moderate risk because it changes CI behavior and can newly fail pull requests based on lint results or git diff detection (shallow fetch/base SHA handling). No runtime/product logic is affected.
Overview
Adds ESLint v9 with a new flat config (
eslint.config.mjs) and new npm scripts (lint:changed,lint:all) to enforce consistent linting for JS/TS.Introduces
scripts/lint-changed.mjs, which determines changed files (PR base vs head in CI, or working tree locally), filters to lintable extensions while ignoring build/artifact paths, and runs ESLint with--max-warnings=0.Updates GitHub Actions to run this lint gate once per pull request as a separate
lintjob (Node 22), instead of relying on the full test matrix to surface lint issues.Reviewed by Cursor Bugbot for commit 086b8e3. Bugbot is set up for automated code reviews on this repo. Configure here.