Skip to content

Conversation

@deanq
Copy link
Contributor

@deanq deanq commented Nov 21, 2025

Changes:

  • Update grep pattern from runpod~= to runpod>=
  • Replace major.minor range logic with semantic version comparison
  • Add validation for current_version extraction
  • Update sed replacement to maintain >= operator
  • Workflow now triggers on any version increase (patch, minor, major)

Previously, the workflow only updated for major.minor changes due to ~= (compatible release) constraint. Now with >=, all new releases are honored.

Changes:
- Update grep pattern from runpod~= to runpod>=
- Replace major.minor range logic with semantic version comparison
- Add validation for current_version extraction
- Update sed replacement to maintain >= operator
- Workflow now triggers on any version increase (patch, minor, major)

Previously, the workflow only updated for major.minor changes due to ~=
(compatible release) constraint. Now with >=, all new releases are honored.
@deanq deanq requested a review from Copilot November 21, 2025 06:23
Copy link

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 updates the CI workflow to adopt semantic versioning with >= instead of compatible release constraints with ~=, enabling the workflow to trigger on all version updates (patch, minor, major) rather than just major.minor changes.

Key changes:

  • Replace ~= (compatible release) with >= operator in requirements.txt and workflow
  • Implement semantic version comparison using sort -V instead of major.minor string matching
  • Add validation for current version extraction

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
builder/requirements.txt Updates runpod dependency from ~=1.7.9 to >=1.8.0 to allow any version >= 1.8.0
.github/workflows/CI-runpod_dep.yml Refactors version comparison logic to use semantic versioning and updates grep/sed patterns to handle >= operator

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

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