You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Hello,
I'd like to implement granular error fixes for VS Code. At the moment the extension just highlights the whole code line with error(s).
Could you validate my implementation plan? The main priorities are being future-proof and easy to maintain.
-fix flag applies all fixes at once, so it doesn't seem to be the best solution for displaying granular errors and fixes in IDE.
The plan:
Run protolint with -reporter json flags (as I haven't found rule field in plain or unix report formats).
Make a regex pattern for every rule, e.g. for FIELD_NAMES_LOWER_SNAKE_CASE:
pattern: /^Field name "(?<wrong>.+)" must be underscore_separated_names like "(?<fix>.+)"$/
Extract granular error ranges and fixes (if available) via these patterns and protolint JSON report. Hopefully, if a rule is fixable, its linter message always contains the fix information.
Thank you!
The text was updated successfully, but these errors were encountered:
Thank you for your interest in the protolint ecosystem!
I was wondering if there is a demand for granular fixes in general and whether it is worth the extra effort to make the protolint error literal an API.
In my opinion, fixing the current file at once, as ale fixer does, is sufficient.
I have already heard that even this simpler and quicker first step has value, as seen in this issue: plexsystems/vscode-protolint#8.
Hello,
I'd like to implement granular error fixes for VS Code. At the moment the extension just highlights the whole code line with error(s).
Could you validate my implementation plan? The main priorities are being future-proof and easy to maintain.
-fix
flag applies all fixes at once, so it doesn't seem to be the best solution for displaying granular errors and fixes in IDE.The plan:
-reporter json
flags (as I haven't foundrule
field in plain or unix report formats).FIELD_NAMES_LOWER_SNAKE_CASE
:pattern: /^Field name "(?<wrong>.+)" must be underscore_separated_names like "(?<fix>.+)"$/
rule
is fixable, its lintermessage
always contains the fix information.Thank you!
The text was updated successfully, but these errors were encountered: