Skip to content

Conversation

@DevelopmentCats
Copy link
Contributor

Description

This pull request improves the version bump script making it more robust for future registry changes.

Type of Change

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

Testing & Validation

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

Related Issues

#510

@DevelopmentCats DevelopmentCats changed the title fix: version-bump script final fix fix: version-bump script fix Nov 18, 2025
Copilot finished reviewing on behalf of DevelopmentCats November 18, 2025 18:27
Copy link
Contributor

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 pull request refactors the version-bump script's AWK logic to improve how it updates version references in README files. Instead of using a stateful approach that tracks whether we're in a target module line-by-line, it now accumulates entire module blocks and processes them as complete units.

  • Replaces line-by-line version update logic with block-based accumulation and processing
  • Adds logic to detect module blocks, accumulate their content, check for target source, and update version only in matching modules
  • Changes the approach from tracking in_target_module state to using module_has_target_source flag after accumulating the full module block

@DevelopmentCats DevelopmentCats merged commit b4c162d into main Nov 20, 2025
4 checks passed
@DevelopmentCats DevelopmentCats deleted the cat/version-bump-update branch November 20, 2025 14:07
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