Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(eslint): Enable eslint rules that overlap with biome #83694

Merged
merged 1 commit into from
Jan 21, 2025

Conversation

ryan953
Copy link
Member

@ryan953 ryan953 commented Jan 18, 2025

A little followup to #83693

This enables 3 rules from biome that are implemented in eslint as part of this plugin.

This plugin has a lot of rules, the recommended set is too opinionated, but a handful would be nice to enable inside sentry. I'll make a list with TODO comments as a followup, and then work against that list rule by rule in followup PRs. For now we just need to pluck these three rules to match what's already happening.

@ryan953 ryan953 requested review from a team as code owners January 18, 2025 04:21
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Jan 18, 2025
@ryan953 ryan953 merged commit 0b216d7 into master Jan 21, 2025
41 checks passed
@ryan953 ryan953 deleted the ryan953/eslint-unicorn branch January 21, 2025 18:58
andrewshie-sentry pushed a commit that referenced this pull request Jan 22, 2025
A little followup to #83693

This enables 3 rules from biome that are implemented in eslint as part
of this plugin.

This plugin has a lot of rules, the recommended set is too opinionated,
but a handful would be nice to enable inside sentry. I'll make a list
with TODO comments as a followup, and then work against that list rule
by rule in followup PRs. For now we just need to pluck these three rules
to match what's already happening.
@github-actions github-actions bot locked and limited conversation to collaborators Feb 6, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
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