Skip to content

lint: Add lint rule for calls to renderHook and derivitives#106492

Open
ryan953 wants to merge 9 commits intomasterfrom
ryan953/lint-renderHook
Open

lint: Add lint rule for calls to renderHook and derivitives#106492
ryan953 wants to merge 9 commits intomasterfrom
ryan953/lint-renderHook

Conversation

@ryan953
Copy link
Member

@ryan953 ryan953 commented Jan 17, 2026

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Jan 17, 2026
@ryan953 ryan953 changed the title Ryan953/lint render hook lint: Add lint rule for calls to renderHook and derivitives Jan 17, 2026
Copy link
Member

@scttcper scttcper left a comment

Choose a reason for hiding this comment

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

nice to point people towards the right thing

@ryan953
Copy link
Member Author

ryan953 commented Jan 17, 2026

nice to point people towards the right thing

@scttcper
Still having issues with N arguments :(
I think we might need a rule before this one that encourages component style {}: Props or something like that.

I paused iterating on it now, if you wanna pick at it.

@ryan953 ryan953 force-pushed the ryan953/lint-renderHook branch from ab9304e to 6a34b8b Compare February 6, 2026 18:51
@ryan953 ryan953 marked this pull request as ready for review February 6, 2026 19:08
@ryan953 ryan953 requested review from a team as code owners February 6, 2026 19:08
Copy link
Contributor

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

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.

2 participants