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

Track line and column number in reported lint location #1019

Open
carolynvs opened this issue Apr 30, 2020 · 0 comments
Open

Track line and column number in reported lint location #1019

carolynvs opened this issue Apr 30, 2020 · 0 comments
Labels
hmm 🛑🤔 Needs more thinking time. Don't start on it yet, please.

Comments

@carolynvs
Copy link
Member

In order for an editor to highlight the location of lint errors, we need to report back the line and column. I think we can achieve that using v3 of the yaml library.

See

We are thinking that this would require passing structured data to the mixins for just the lint command that contains the line data, maybe more. This would need a spike to figure out how to do this isolating Porter from the Node structs, and not requiring mixins to reinvent the wheel over and over, or have to use Go.

Manifest wide will want linting, this isn't just for the mixins.

@carolynvs carolynvs added the hmm 🛑🤔 Needs more thinking time. Don't start on it yet, please. label Apr 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hmm 🛑🤔 Needs more thinking time. Don't start on it yet, please.
Projects
None yet
Development

No branches or pull requests

1 participant