Skip to content

Release v0.6.5#38

Merged
thomashebrard merged 1 commit into
mainfrom
release/v0.6.5
Nov 27, 2025
Merged

Release v0.6.5#38
thomashebrard merged 1 commit into
mainfrom
release/v0.6.5

Conversation

@thomashebrard
Copy link
Copy Markdown
Member

No description provided.

@thomashebrard thomashebrard changed the title release/v0.6.5 Release v0.6.5 Nov 27, 2025
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines 67 to 70
# Check if versions match
if [[ "$PR_VERSION" != "$RELEASE_VERSION" ]]; then
echo "❌ ERROR: Version in pyproject.toml ($PR_VERSION) does not match source release branch version ($RELEASE_VERSION)"
echo "❌ ERROR: Version in pyproject.toml ($PR_VERSION) does not match release branch version ($RELEASE_VERSION)"
exit 1
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Version check fails all non-release PRs

When the source branch is not named like release/v*, the Get branch info step exits early after printing “skipping version check” and never populates source_release_version, but the Check version matches release branch block still runs unconditionally and compares the pyproject version against that empty string, triggering the exit-1 path. As a result every normal PR to main now fails this workflow instead of being skipped as intended.

Useful? React with 👍 / 👎.

Copy link
Copy Markdown
Member

@lchoquel lchoquel left a comment

Choose a reason for hiding this comment

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

OK

@thomashebrard thomashebrard merged commit 33417f0 into main Nov 27, 2025
19 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Nov 27, 2025
@thomashebrard thomashebrard deleted the release/v0.6.5 branch December 1, 2025 11:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants