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:16
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 refactors the CI workflow to adopt semantic versioning with >= instead of the compatible release ~= operator for the runpod dependency. This change allows the workflow to automatically update for any new version (patch, minor, or major) rather than only major.minor changes.

Key Changes:

  • Updated version constraint operator from ~= to >= in requirements.txt
  • Replaced major.minor range comparison with full semantic version comparison
  • Added validation for current version extraction to prevent silent failures

Reviewed Changes

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

File Description
builder/requirements.txt Updated runpod version constraint from ~=1.7.0 to >=1.8.0
.github/workflows/CI-runpod_dep.yml Refactored version comparison logic to support >= operator and added current version extraction validation

💡 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