Skip to content

Conversation

DevelopmentCats
Copy link
Contributor

Description

Update version detection to always detect named module block, and extract version from same module block.
Ensure that script is completely compatible for all Unix environments.

Type of Change

  • New module
  • Bug fix
  • Feature/enhancement
  • Documentation
  • Other

Testing & Validation

  • Tests pass (bun test)
  • Code formatted (bun run fmt)
  • Changes tested locally

@Copilot Copilot AI review requested due to automatic review settings August 18, 2025 19:14
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR fixes version extraction to be more robust by refactoring the version detection logic to ensure it extracts version information from the correct Terraform module block and improves Unix shell compatibility.

  • Refactored version extraction to use a dedicated function that searches within specific module blocks
  • Removed unused exit code constant and updated documentation
  • Fixed shell compatibility issues with variable comparisons

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@DevelopmentCats DevelopmentCats merged commit 858799c into main Aug 18, 2025
4 checks passed
@DevelopmentCats DevelopmentCats deleted the cat/tagrelease-fix branch August 18, 2025 19:36
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.

2 participants