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

Fixing bicep build error with linter in Bicep CLI v0.10.61 #400

Merged
merged 1 commit into from
Sep 14, 2022
Merged

Conversation

Gordonby
Copy link
Collaborator

@Gordonby Gordonby commented Sep 14, 2022

PR Summary

New linting rule brought in in v0.10.61 causes our IaC to error out.
Rather than change our linting rules, i've refactored the bicep to comply with the rule.

Closes #399

PR Checklist

  • PR has a meaningful title
  • This PR is ready to merge and is not Work in Progress
  • Link to a filed issue
  • Summarized changes

Signed-off-by: Gordonby <gordon.byers@microsoft.com>
@Gordonby Gordonby temporarily deployed to azurecirgs September 14, 2022 06:35 Inactive
@Gordonby Gordonby enabled auto-merge (squash) September 14, 2022 06:35
@Gordonby Gordonby self-assigned this Sep 14, 2022
@Gordonby Gordonby changed the title Fixing bicep build error with Bicep CLI v0.10.61 Fixing bicep build error with linter in Bicep CLI v0.10.61 Sep 14, 2022
@Gordonby Gordonby added the bug Something isn't working label Sep 14, 2022
@Gordonby Gordonby requested a review from a team September 14, 2022 15:16
@Gordonby Gordonby merged commit 8f0dae4 into main Sep 14, 2022
@Gordonby Gordonby deleted the gb-BCP270 branch September 14, 2022 15:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bicep v0.10.61 linter errors on conditional scopes
2 participants