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

perf(linter/react): find class node by symbols in get_parent_es6_component #1657

Conversation

Dunqing
Copy link
Member

@Dunqing Dunqing commented Dec 12, 2023

This way we can get the class node faster. But I don't know if this is a good way. In eslint-plugin-react, they get class node by scope. But oxc cannot do the same way

@Dunqing
Copy link
Member Author

Dunqing commented Dec 12, 2023

Current dependencies on/for this PR:

This stack of pull requests is managed by Graphite.

@github-actions github-actions bot added A-linter Area - Linter A-semantic Area - Semantic labels Dec 12, 2023
Copy link
Member

@Boshen Boshen left a comment

Choose a reason for hiding this comment

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

How did you measure the performance of this change?

crates/oxc_semantic/src/symbol.rs Show resolved Hide resolved
@Boshen Boshen merged commit 0080638 into main Dec 13, 2023
17 checks passed
@Boshen Boshen deleted the 12-12-perf_linter/react_find_class_node_by_symbols_in_get_parent_es6_component branch December 13, 2023 05:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-linter Area - Linter A-semantic Area - Semantic
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants