Skip to content

Pin terraform-config-inspect version - #2197

Merged
theseanything merged 1 commit into
mainfrom
fix/pin-terraform-config-inspect
Jul 10, 2026
Merged

Pin terraform-config-inspect version#2197
theseanything merged 1 commit into
mainfrom
fix/pin-terraform-config-inspect

Conversation

@theseanything

Copy link
Copy Markdown
Contributor

mise now applies a 24h minimum_release_age filter by default, which has broken CI for this repository.

This module has no tagged releases, so "latest" resolves to nothing whenever the newest upstream commit is under 24h old, breaking Terraform CI. Pinning an exact version bypasses the filter and makes CI reproducible.

mise now applies a 24h minimum_release_age filter by default. This
module has no tagged releases, only commit pseudo-versions, so
"latest" resolves to nothing whenever the newest upstream commit is
under 24h old, breaking Terraform CI. Pinning an exact version
bypasses the filter and makes CI reproducible.
Copilot AI review requested due to automatic review settings July 10, 2026 12:40

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Pins the terraform-config-inspect Go tool in .mise.toml to an explicit Go pseudo-version to avoid mise’s default minimum_release_age filter breaking CI when latest points at a very recent upstream commit.

Changes:

  • Replace latest for go:github.com/hashicorp/terraform-config-inspect with a pinned commit pseudo-version.
  • Add inline documentation explaining the CI failure mode and where to find the current upstream “@latest” pseudo-version.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@whi-tw whi-tw left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

alternative approach might be to use use the minimum_release_age_excludes for this dependency, but this way at least we're pinned to something vaguely sensible.

@theseanything
theseanything added this pull request to the merge queue Jul 10, 2026
Merged via the queue into main with commit 9b38310 Jul 10, 2026
19 checks passed
@theseanything
theseanything deleted the fix/pin-terraform-config-inspect branch July 10, 2026 12:50
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.

3 participants