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.

Sign up for GitHub

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

Jump to bottom

📎 Biome should report the location of the error in an invalid configuration file #1627

Closed
Conaclos opened this issue Jan 21, 2024 · 0 comments · Fixed by #2284
Closed

📎 Biome should report the location of the error in an invalid configuration file #1627

Conaclos opened this issue Jan 21, 2024 · 0 comments · Fixed by #2284
Assignees
@Conaclos
Labels
A-Parser Area: parser L-JSON Language: JSON and super languages S-Enhancement Status: Improve an existing feature S-Help-wanted Status: you're familiar with the code base and want to help the project

Comments

@Conaclos
Copy link
Member

Description

When Biome encounters a malformed configuration, it errors without giving the location of the error. For large configuration file, this can be hard to locate the error. Biome should report the location of the error.

❯ cat biome.json 
{
    linter: {}
}

❯ npx biome lint index.js 
/playroom/prettier/biome.json parse ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

  ✖ Property key must be double quoted

configuration ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

  ✖ Biome exited because the configuration resulted in errors. Please fix them.
@Conaclos Conaclos added A-Parser Area: parser L-JSON Language: JSON and super languages S-Enhancement Status: Improve an existing feature S-Help-wanted Status: you're familiar with the code base and want to help the project labels Jan 21, 2024
@Conaclos Conaclos self-assigned this Apr 3, 2024
@Conaclos Conaclos mentioned this issue Apr 3, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Conaclos Conaclos

Labels
A-Parser Area: parser L-JSON Language: JSON and super languages S-Enhancement Status: Improve an existing feature S-Help-wanted Status: you're familiar with the code base and want to help the project
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

refactor(biome_deserialize): display parsing error source biomejs/biome
1 participant
@Conaclos