Skip to content

feat(eslint): no-redundant default argument - #122820

Open
TkDodo wants to merge 8 commits into
masterfrom
tkdodo/feat/eslint-no-redundant-default-argument
Open

feat(eslint): no-redundant default argument#122820
TkDodo wants to merge 8 commits into
masterfrom
tkdodo/feat/eslint-no-redundant-default-argument

Conversation

@TkDodo

@TkDodo TkDodo commented Aug 27, 2026

Copy link
Copy Markdown
Collaborator

add a new, auto-fixable lint rule that stops us from passing redundant default values because they are the same literal value:

Screenshot 2026-08-27 at 11 13 57

@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Aug 27, 2026
@TkDodo

TkDodo commented Aug 27, 2026

Copy link
Copy Markdown
Collaborator Author

bugbot run

@github-actions

github-actions Bot commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

📊 Type Coverage Diff

✅ no issues found

Comment thread static/eslint/eslintPluginSentry/noRedundantDefaultArgument.ts
@TkDodo

TkDodo commented Aug 27, 2026

Copy link
Copy Markdown
Collaborator Author

bugbot run

@github-actions

Copy link
Copy Markdown
Contributor

@cursor cursor Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit c544fb5. Configure here.

@TkDodo
TkDodo marked this pull request as ready for review August 27, 2026 10:52
@TkDodo
TkDodo requested review from a team as code owners August 27, 2026 10:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant