AdguardTeam / AdguardTeam/AGLint

Extend the config validator to list issues with locations

Ouverte
#246 0 commentaires 0 réactions 1 personne assignée Réclamée par @maximtop Voir sur GitHub
Priority: P4 T: Config
Langage dominant
TypeScript
Étoiles
71
Forks
4
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

Currently, the config validator throws an error if the config is invalid. However, we can enhance it by assigning exact locations to the errors and linking these locations to specific error messages. This would enable the VSCode extension to highlight configuration issues directly in the editor. Reference: https://github.com/AdguardTeam/VscodeAdblockSyntax/issues/146

For this, it may be necessary to replace the YAML parser with the following one: [https://www.npmjs.com/package/yaml](https://www.npmjs.com/package/yaml).

For parsing JSON with locations, we should take a look at https://www.npmjs.com/package/@humanwhocodes/momoa or https://www.npmjs.com/package/json-to-ast

Guide de contribution

Ouvrir le guide de contribution

Évaluation

Cette issue n'a pas encore été évaluée.

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.