chore: add ratchet rules to stop nesting components #42962
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
7 Skipped Deployments
|
|
This pull request has been ignored for the connected project Preview Branches by Supabase. |
|
No actionable comments were generated in the recent review. 🎉 📝 WalkthroughSummary by CodeRabbit
WalkthroughThis PR enables the react/no-unstable-nested-components rule in the studio ESLint config and ratchet script, and records its occurrences in apps/studio/.github/eslint-rule-baselines.json (including a new ruleFiles.no-restricted-imports mapping). jsx-a11y/alt-text is added as a warning in the ESLint config. The lint:ratchet npm script in apps/studio/package.json is updated to include the new rule. A small change in AuthProvidersForm/FormField.tsx adds alt attributes to rendered images and reorders imports. Sequence Diagram(s)(omitted) Assessment against linked issues
Out-of-scope changes
Comment |
🎭 Playwright Test ResultsDetails
Flaky testsFeatures › cron-jobs.spec.ts › Cron Jobs Integration › Cron Jobs CRUD Operations › can view cron jobs page Skipped testsFeatures › sql-editor.spec.ts › SQL Editor › snippet favourite works as expected |
…sgrid-out-of-integrationspage
…sgrid-out-of-integrationspage
…sgrid-out-of-integrationspage
Braintrust eval report
|
I have read the CONTRIBUTING.md file.
YES
What kind of change does this PR introduce?
The changes required for this React Doctor need more thought that I can hammer with AI. Just need to stop this from happening in the future with Ratchet rules