Skip to content

[workspace/orchestrator] chore(release): fix typo that results in is_version_pr being falsely evaluated - #2469

Merged
hopehadfield merged 1 commit into
redhat-developer:workspace/orchestratorfrom
openshift-cherrypick-robot:cherry-pick-2468-to-workspace/orchestrator
Mar 5, 2026
Merged

[workspace/orchestrator] chore(release): fix typo that results in is_version_pr being falsely evaluated#2469
hopehadfield merged 1 commit into
redhat-developer:workspace/orchestratorfrom
openshift-cherrypick-robot:cherry-pick-2468-to-workspace/orchestrator

Conversation

@openshift-cherrypick-robot

Copy link
Copy Markdown

This is an automated cherry-pick of #2468

/assign hopehadfield

…evaluated

Signed-off-by: Hope Hadfield <hhadfiel@redhat.com>
@rhdh-qodo-merge

Copy link
Copy Markdown

Review Summary by Qodo

Fix variable reference typo in release workflow

🐞 Bug fix

Grey Divider

Walkthroughs

Description
• Fix typo in release workflow variable reference
• Missing $ prefix caused PR_MERGED variable to be evaluated as literal string
• Corrects conditional logic for version PR detection
Diagram
flowchart LR
  A["Release Workflow"] -->|"Checks PR_MERGED variable"| B["Before: Literal 'PR_MERGED' string"]
  A -->|"Fixed with $ prefix"| C["After: Actual variable value"]
  B -->|"Always false"| D["Incorrect version PR detection"]
  C -->|"Correct evaluation"| E["Proper version PR detection"]
Loading

Grey Divider

File Changes

1. .github/workflows/release_workspace_version.yml 🐞 Bug fix +1/-1

Fix PR_MERGED variable reference in conditional

• Fixed typo in conditional check: changed "PR_MERGED" to "$PR_MERGED"
• Missing dollar sign prefix caused variable to be treated as literal string instead of being
 evaluated
• Corrects the is_version_pr output determination logic

.github/workflows/release_workspace_version.yml


Grey Divider

Qodo Logo

@rhdh-qodo-merge

rhdh-qodo-merge Bot commented Mar 5, 2026

Copy link
Copy Markdown

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

ⓘ The new review experience is currently in Beta. Learn more

Grey Divider

Qodo Logo

@sonarqubecloud

sonarqubecloud Bot commented Mar 5, 2026

Copy link
Copy Markdown

@hopehadfield
hopehadfield merged commit 11e7af8 into redhat-developer:workspace/orchestrator Mar 5, 2026
2 checks passed
hopehadfield added a commit to hopehadfield/rhdh-plugins that referenced this pull request Mar 5, 2026
…evaluated (redhat-developer#2469)

Signed-off-by: Hope Hadfield <hhadfiel@redhat.com>
Co-authored-by: Hope Hadfield <hhadfiel@redhat.com>
hopehadfield added a commit that referenced this pull request Mar 5, 2026
…evaluated (#2469) (#2470)

Signed-off-by: Hope Hadfield <hhadfiel@redhat.com>
Co-authored-by: OpenShift Cherrypick Robot <openshift-cherrypick-robot@redhat.com>
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