Skip to content

Conversation

@yhakbar
Copy link
Collaborator

@yhakbar yhakbar commented Nov 5, 2025

Description

Had to temporarily pin this to a commit before, and this restores it to a tag-based commit:
7c30c1b

TODOs

Read the Gruntwork contribution guidelines.

  • I authored this code entirely myself
  • I am submitting code based on open source software (e.g. MIT, MPL-2.0, Apache)]
  • I am adding or upgrading a dependency or adapted code and confirm it has a compatible open source license
  • Update the docs.
  • Run the relevant tests successfully, including pre-commit checks.
  • Include release notes. If this PR is backward incompatible, include a migration guide.

Release Notes (draft)

Updated commit-based remote references to tag-based references.

Migration Guide

Summary by CodeRabbit

  • Chores
    • Updated test fixture references to use a tagged release version (v0.93.2) instead of specific commit hashes for improved consistency and maintainability.

@vercel
Copy link

vercel bot commented Nov 5, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
terragrunt-docs Ready Ready Preview Comment Nov 5, 2025 1:58pm

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 5, 2025

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

Test fixture files in the download directory have been updated to reference module version v0.93.2 via tag instead of a specific commit hash (2ca67fe). This changes which module version is retrieved during test execution.

Changes

Cohort / File(s) Summary
Test fixture version reference updates
test/fixtures/download/hello-world/main.tf, test/fixtures/download/remote/terragrunt.hcl, test/fixtures/download/remote-ref/terragrunt.hcl, test/fixtures/download/remote-relative/terragrunt.hcl, test/fixtures/download/remote-relative-with-slash/terragrunt.hcl
Updated module source references from commit SHA 2ca67fe2dbd3001c4cffa66df9567ca497182cb1 to tag v0.93.2 across all download test fixtures

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

  • Straightforward, repetitive version reference updates across test fixtures
  • No logic changes, control flow modifications, or new functionality introduced
  • Consistent pattern applied uniformly across five files

Possibly related PRs

Suggested reviewers

  • denis256
  • levkohimins
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore/update-pin-for-fixtures

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between dd7913e and 0db776b.

📒 Files selected for processing (5)
  • test/fixtures/download/hello-world/main.tf (1 hunks)
  • test/fixtures/download/remote-ref/terragrunt.hcl (1 hunks)
  • test/fixtures/download/remote-relative-with-slash/terragrunt.hcl (1 hunks)
  • test/fixtures/download/remote-relative/terragrunt.hcl (1 hunks)
  • test/fixtures/download/remote/terragrunt.hcl (1 hunks)

Comment @coderabbitai help to get the list of available commands and usage tips.

@yhakbar yhakbar marked this pull request as ready for review November 5, 2025 15:10
@yhakbar yhakbar requested a review from denis256 as a code owner November 5, 2025 15:10
@yhakbar yhakbar enabled auto-merge (squash) November 5, 2025 15:11
@yhakbar yhakbar merged commit 8f2edbb into main Nov 5, 2025
52 of 54 checks passed
@yhakbar yhakbar deleted the chore/update-pin-for-fixtures branch November 5, 2025 15:12
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