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

feat(task): init eslint-plugin-tree-shaking rule #2662

Merged
merged 1 commit into from Mar 10, 2024

Conversation

mysteryven
Copy link
Member

No description provided.

Copy link
Member Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @mysteryven and the rest of your teammates on Graphite Graphite

@github-actions github-actions bot added the A-linter Area - Linter label Mar 10, 2024
@mysteryven mysteryven force-pushed the 03-10-feat_task_init_eslint-plugin-tree-shaking_rule branch from a7e5ed0 to 1c39e64 Compare March 10, 2024 11:34
Copy link

codspeed-hq bot commented Mar 10, 2024

CodSpeed Performance Report

Merging #2662 will not alter performance

Comparing 03-10-feat_task_init_eslint-plugin-tree-shaking_rule (9047964) with main (a218242)

Summary

✅ 29 untouched benchmarks

@mysteryven mysteryven force-pushed the 03-10-feat_task_init_eslint-plugin-tree-shaking_rule branch from 1c39e64 to bfade28 Compare March 10, 2024 13:04
@mysteryven mysteryven force-pushed the 03-10-feat_task_init_eslint-plugin-tree-shaking_rule branch from bfade28 to 9047964 Compare March 10, 2024 13:09
Copy link
Member

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.

Copy link
Member

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:
Copy link
Member

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.

Copy link
Member

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.

@Boshen Boshen enabled auto-merge (squash) March 10, 2024 14:07
@Boshen Boshen disabled auto-merge March 10, 2024 14:07
@Boshen Boshen merged commit f8e8af2 into main Mar 10, 2024
23 checks passed
@Boshen Boshen deleted the 03-10-feat_task_init_eslint-plugin-tree-shaking_rule branch March 10, 2024 14:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-linter Area - Linter
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants