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

The linter ruleset should be versionable #4719

Open
alex-frankel opened this issue Oct 4, 2021 · 3 comments
Open

The linter ruleset should be versionable #4719

alex-frankel opened this issue Oct 4, 2021 · 3 comments
Assignees
Labels
devdiv Related to Bicep tooling efforts in DevDiv enhancement New feature or request story: linter

Comments

@alex-frankel
Copy link
Collaborator

alex-frankel commented Oct 4, 2021

At some point we will introduce the ability to treat warnings as errors. At that point, if we add a new linting rule it could be a breaking change, so we should have the ability to pin the linter rule set to a specific version in bicepconfig.json. Something like:

{
  "analyzers": {
    "core": {
      "ruleSetVersion": "x.y.z"
      ...
    }
  }
}

cc @MarcusFelling @StephenWeatherford

@alex-frankel alex-frankel added the enhancement New feature or request label Oct 4, 2021
@ghost ghost added the Needs: Triage 🔍 label Oct 4, 2021
@MarcusFelling
Copy link
Collaborator

Related to #2811

@StephenWeatherford
Copy link
Contributor

Whatever mechanism is used, ideally Intellisense for rule names (at least core ones) will work based on the rules available only in the specified ruleset version.

@StephenWeatherford
Copy link
Contributor

#8013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
devdiv Related to Bicep tooling efforts in DevDiv enhancement New feature or request story: linter
Projects
None yet
Development

No branches or pull requests

3 participants