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
improvement(linter/oxc): using facades for barrel detections. #3215
improvement(linter/oxc): using facades for barrel detections. #3215
Conversation
CodSpeed Performance ReportMerging #3215 will not alter performanceComparing Summary
|
bdb6fb6
to
9e07df8
Compare
This stack of pull requests is managed by Graphite. Learn more about stacking. |
|
||
fn run(&self, node: &AstNode<'_>, ctx: &LintContext<'_>) { | ||
let AstKind::ImportDeclaration(import) = node.kind() else { return }; | ||
if is_facade_import(ctx.file_path().to_str().unwrap(), import.source.value.as_str()) { |
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 can simplify this, leave this to me.
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.
Feel free to take over this PR, I'll start working on other stuff. Let me know if there is anything you want me to change.
Now I understand what this is doing when it's written in Rust 😅 |
Closed in favor of #3275 |
closes #3069
It is similar to this, I've dialed back some package resolving to make it more general, We might want to invest in creating a package resolver(let me know if we already have the infrastructure for it and I've missed it).