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

linter: no-restricted-imports missing option support for allowTypeImports, allowImportNames, group, .. #7810

Open
Sysix opened this issue Dec 12, 2024 · 1 comment
Assignees
Labels
A-linter Area - Linter C-bug Category - Bug

Comments

@Sysix
Copy link
Contributor

Sysix commented Dec 12, 2024

What version of Oxlint are you using?

0.15.0

What command did you run?

--

What does your .oxlint.json config file look like?


What happened?

Because this is originally only a eslint rule, it does not support the option of the alias rule @typescript-eslint/no-restricted-imports.

See the implementation here:

struct RestrictedPath {
name: CompactStr,
#[serde(rename = "importNames")]
import_names: Option<Box<[CompactStr]>>,
message: Option<CompactStr>,
}

I expect to use allowTypeImports option

@Sysix Sysix added C-bug Category - Bug A-linter Area - Linter labels Dec 12, 2024
@Sysix Sysix self-assigned this Dec 14, 2024
@Sysix
Copy link
Contributor Author

Sysix commented Dec 14, 2024

It is also missing the eslint option allowImportNames , group, regex, caseSensitive and many more >.>

@Sysix Sysix changed the title linter: typescript/no-restricted-imports missing option support for allowTypeImports linter: no-restricted-imports missing option support for allowTypeImports, allowImportNames, group, .. Dec 14, 2024
Boshen pushed a commit that referenced this issue Dec 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-linter Area - Linter C-bug Category - Bug
Projects
None yet
Development

No branches or pull requests

1 participant