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

Consider supporting autofix. #374

Open
ryaneberly opened this issue Jul 28, 2017 · 1 comment
Open

Consider supporting autofix. #374

ryaneberly opened this issue Jul 28, 2017 · 1 comment
Milestone

Comments

@ryaneberly
Copy link
Contributor

---- from @denuno
That would be a pretty interesting thing to have in CFLint itself, actually, along with a mode where you could enable auto-fixing (or maybe auto-ignoring)... e.g. instead of just warning about an unscoped var, optionally have it var scope it... yeah, something like enabling "suggestions", where it says what rule is broke and then also (if it is easy to figure out) shows a proposed fix.

Something to think about at least. For sure would want it to be optional, as something like that would slow things down a bit I reckon.

@KamasamaK
Copy link
Collaborator

For this to work, you need accurate locations/ranges. I have implemented this myself in my VS Code extension for some of the simple rules, but it doesn't always work as expected due to inaccurate location reporting.

Blockers: #570, #522

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants