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

Feature: Support svlint v0.8.0 with TextRule and SyntaxRule. #216

Merged
merged 8 commits into from
Aug 10, 2023

Conversation

DaveMcEwan
Copy link
Contributor

@DaveMcEwan DaveMcEwan commented Jun 26, 2023

Support the new API of svlint v0.8.0 v0.9.0 with both TextRule and SyntaxRule.
This PR matches the work in dalance/svlint#253, dalance/svlint#255, dalance/svlint-plugin-sample#7, and dalance/svlint-plugin-sample#8.

Additional minor things:

  • Put a debug!() message into svls.log when Backend::lint() fails to initialize linter.
  • Delete the checked-in svls.log.
  • Fixes the example in Support for different linter command #192.
  • Allow use of environment variable SVLS_CONFIG, just like SVLINT_CONFIG.
  • Support svlint plugins via .svls.toml.

- Makes it easier to debug/dev svls.
- Allows use of wrapper scripts for multiple configs on the same repo.
@DaveMcEwan
Copy link
Contributor Author

Updated to match new release of svlint (v0.9.0).

@dalance
Copy link
Owner

dalance commented Aug 10, 2023

Thanks!

@dalance dalance merged commit 13ed009 into dalance:master Aug 10, 2023
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants