Skip to content

ci,fix: Pin version of bicep-cli used by pre-commit#219

Merged
kdestin merged 1 commit intoAzure-Samples:mainfrom
kdestin:fix-agent-bicep-build-pre-commit
Mar 31, 2025
Merged

ci,fix: Pin version of bicep-cli used by pre-commit#219
kdestin merged 1 commit intoAzure-Samples:mainfrom
kdestin:fix-agent-bicep-build-pre-commit

Conversation

@kdestin
Copy link
Copy Markdown
Member

@kdestin kdestin commented Mar 31, 2025

Description

Bicep includes its version and a hash of the template as metadata
in the file produced when running `bicep build`. Different versions
of bicep-cli produce differing values for those fields, which
can break `pre-commit run --all` when run against changes that
don't touch any bicep files.

Pinning bicep-cli should keep those fields consistent until
bicep is intentionally upgraded.

Checklist

  • I have read the contribution guidelines
  • I have coordinated with the docs team (mldocs@microsoft.com) if this PR deletes files or changes any file names or file extensions.
  • This notebook or file is added to the CODEOWNERS file, pointing to the author or the author's team.

    Bicep includes its version and a hash of the template as metadata
    in the file produced when running `bicep build`. Different versions
    of bicep-cli produce differing values for those fields, which
    can break `pre-commit run --all` when run against changes that
    don't touch any bicep files.

    Pinning bicep-cli should keep those fields consistent until
    bicep is intentionally upgraded.
@kdestin kdestin requested a review from a team as a code owner March 31, 2025 19:01
@kdestin kdestin enabled auto-merge (squash) March 31, 2025 19:02
@kdestin kdestin disabled auto-merge March 31, 2025 19:05
@kdestin kdestin merged commit c65354c into Azure-Samples:main Mar 31, 2025
3 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.

1 participant