Skip to content

chore: quiet lint gate output - #49

Merged
edhor1608 merged 1 commit into
mainfrom
repo-refresh-lint-gate-cleanup
May 2, 2026
Merged

edhor1608 merged 1 commit into
mainfrom
repo-refresh-lint-gate-cleanup

Conversation

@edhor1608

@edhor1608 edhor1608 commented May 1, 2026 •

Copy link
Copy Markdown
Owner

Quiet lint gate output by scoping lint inputs and fixing lint warnings

  • Scopes the lint script in package.json to src, tests, config, package.json, and tsconfig.json instead of the entire repo, reducing noise.
  • Replaces Array.findIndex with Array.indexOf for locating '--out' across several e2e fixture files.
  • Replaces non-null assertions with nullish coalescing (?? 0) in tests/helpers/image.ts, preventing NaN propagation in diffRgb when channel values are undefined.
  • Uses optional chaining in tests/helpers/cli.ts parseJsonStdout to simplify the last-line check.

Macroscope summarized b99eaa9.

@coderabbitai

coderabbitai Bot commented May 1, 2026 •

Copy link
Copy Markdown

Warning

Rate limit exceeded

@edhor1608 has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 7 minutes and 13 seconds before requesting another review.

To keep reviews running without waiting, you can enable usage-based add-on for your organization. This allows additional reviews beyond the hourly cap. Account admins can enable it under billing.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 62c44e6c-02c4-4d35-9f66-ff423cd29621

📥 Commits

Reviewing files that changed from the base of the PR and between 6643fa8 and b99eaa9.

📒 Files selected for processing (8)
  • docs/repo_refresh_audit.md
  • package.json
  • tests/fixtures/e2e/generate-benchmark-snapshots.ts
  • tests/fixtures/e2e/generate-export-snapshots.ts
  • tests/fixtures/e2e/generate-export-video-snapshots.ts
  • tests/fixtures/e2e/generate-report-export-snapshots.ts
  • tests/helpers/cli.ts
  • tests/helpers/image.ts
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch repo-refresh-lint-gate-cleanup
✨ Simplify code
  • Create PR with simplified code
  • Commit simplified code in branch repo-refresh-lint-gate-cleanup

Review rate limit: 0/5 reviews remaining, refill in 7 minutes and 13 seconds.

Comment @coderabbitai help to get the list of available commands and usage tips.

@edhor1608
edhor1608 force-pushed the repo-refresh-temp-test-outputs branch from 8a4cd36 to fd21e00 Compare May 2, 2026 19:03
@edhor1608
edhor1608 force-pushed the repo-refresh-lint-gate-cleanup branch from 2ace614 to 0425315 Compare May 2, 2026 19:03
@edhor1608
edhor1608 marked this pull request as ready for review May 2, 2026 19:03
@edhor1608
edhor1608 changed the base branch from repo-refresh-temp-test-outputs to main May 2, 2026 21:21
@edhor1608
edhor1608 changed the base branch from main to graphite-base/49 May 2, 2026 21:25
@edhor1608
edhor1608 force-pushed the repo-refresh-lint-gate-cleanup branch from 0425315 to ea72cc0 Compare May 2, 2026 21:25
@edhor1608
edhor1608 changed the base branch from graphite-base/49 to repo-refresh-temp-test-outputs May 2, 2026 21:25
@edhor1608
edhor1608 force-pushed the repo-refresh-lint-gate-cleanup branch 3 times, most recently from ea72cc0 to a5252e9 Compare May 2, 2026 21:32
@edhor1608
edhor1608 force-pushed the repo-refresh-temp-test-outputs branch from 9cd3bf9 to 65dd9ca Compare May 2, 2026 21:32
@graphite-app
graphite-app Bot changed the base branch from repo-refresh-temp-test-outputs to graphite-base/49 May 2, 2026 21:37
@edhor1608
edhor1608 force-pushed the graphite-base/49 branch from 65dd9ca to 6643fa8 Compare May 2, 2026 21:38
@edhor1608
edhor1608 force-pushed the repo-refresh-lint-gate-cleanup branch from a5252e9 to b99eaa9 Compare May 2, 2026 21:38
@graphite-app
graphite-app Bot changed the base branch from graphite-base/49 to main May 2, 2026 21:39
@graphite-app

graphite-app Bot commented May 2, 2026

Copy link
Copy Markdown

Merge activity

  • May 2, 9:39 PM UTC: Graphite rebased this pull request, because this pull request is set to merge when ready.

@edhor1608
edhor1608 merged commit 9742f03 into main May 2, 2026
4 checks passed
@edhor1608
edhor1608 deleted the repo-refresh-lint-gate-cleanup branch May 15, 2026 18:54
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