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 - support different output formats #1462

Closed
fbuchlak opened this issue Nov 20, 2023 · 2 comments · Fixed by #3039
Closed

linter - support different output formats #1462

fbuchlak opened this issue Nov 20, 2023 · 2 comments · Fixed by #3039
Assignees
Labels
A-linter Area - Linter

Comments

@fbuchlak
Copy link

It would be nice to have different output from linter (at least json). This feature would allow developers to integrate this tool easily.

See https://eslint.org/docs/latest/use/formatters/

@Boshen Boshen added A-linter Area - Linter E-Help Wanted Experience level - For the experienced collaborators good first issue Experience Level - Good for newcomers labels Nov 21, 2023
@lukasMega
Copy link

Hello, I would be very grateful for the unix format here. It's simple single-line output. https://eslint.org/docs/latest/use/formatters/#unix
In IntelliJ Idea are these filepaths clickable for one-click opening:

see - Left side: ESLint output (format `unix`) - filepaths are clickable. Right side: oxlint output (default) and filepaths are not clickable :/ Screen 2023-12-18 at 17 59 54

@Boshen Boshen removed E-Help Wanted Experience level - For the experienced collaborators good first issue Experience Level - Good for newcomers labels Jan 6, 2024
@fbuchlak fbuchlak changed the title linter - support different ouput formats linter - support different output formats Feb 16, 2024
@Boshen
Copy link
Member

Boshen commented Feb 20, 2024

The infrastructure is ready to accept more reporters. See https://github.com/oxc-project/oxc/blob/main/crates/oxc_diagnostics/src/reporter.rs

The json reporter does not conform to the ESLint format yet.

PRs are welcomed for both changes.

@Boshen Boshen self-assigned this Mar 4, 2024
Boshen added a commit that referenced this issue Apr 20, 2024
Boshen added a commit that referenced this issue Apr 20, 2024
Boshen added a commit that referenced this issue Apr 20, 2024
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 a pull request may close this issue.

3 participants