-
-
Notifications
You must be signed in to change notification settings - Fork 355
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
feat(task): init eslint-plugin-tree-shaking rule #2662
feat(task): init eslint-plugin-tree-shaking rule #2662
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. Join @mysteryven and the rest of your teammates on Graphite |
a7e5ed0
to
1c39e64
Compare
CodSpeed Performance ReportMerging #2662 will not alter performanceComparing Summary
|
1c39e64
to
bfade28
Compare
bfade28
to
9047964
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we should put this file in the new PR.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is fine, let's merge and iterate.
@@ -117,6 +117,9 @@ new-react-perf-rule name: | |||
new-n-rule name: | |||
cargo run -p rulegen {{name}} n | |||
|
|||
new-tree-shaking-rule name: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We don't need this, there is only one rule.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is fine, let's merge and iterate.
No description provided.