Skip to content

fix ocm release on tags#46

Merged
cnvergence merged 1 commit intomainfrom
fix-ocm-tag-release
Oct 6, 2025
Merged

fix ocm release on tags#46
cnvergence merged 1 commit intomainfrom
fix-ocm-tag-release

Conversation

@cnvergence
Copy link
Copy Markdown
Collaborator

@cnvergence cnvergence commented Oct 3, 2025

Summary by CodeRabbit

  • New Features

    • No user-facing features added in this release.
  • Chores

    • Updated release automation to run without a main-branch restriction, enabling releases from additional branches or tags as needed.
    • Retains existing configuration, credentials, and parameters for consistency.
    • This change improves operational flexibility for releases while having no impact on application behavior, performance, or UI.

Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>

On-behalf-of: @SAP karol.szwaj@sap.com
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Oct 3, 2025

Walkthrough

Removed the branch guard condition from the release job in .github/workflows/job-ocm.yaml, allowing the job to run without checking for refs/heads/main. No other workflow steps or parameters were changed.

Changes

Cohort / File(s) Summary
CI workflow
\.github/workflows/job-ocm.yaml
Deleted the conditional branch check that restricted the release job to refs/heads/main; all other configuration remains unchanged.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes


📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Free

📥 Commits

Reviewing files that changed from the base of the PR and between 3913789 and 7589e47.

📒 Files selected for processing (1)
  • .github/workflows/job-ocm.yaml (0 hunks)
💤 Files with no reviewable changes (1)
  • .github/workflows/job-ocm.yaml

Note

🎁 Summarized by CodeRabbit Free

The PR author is not assigned a seat. To perform a comprehensive line-by-line review, please assign a seat to the pull request author through the subscription management page by visiting https://app.coderabbit.ai/login.

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Contributor

@mirzakopic mirzakopic left a comment

Choose a reason for hiding this comment

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

lgtm

@cnvergence cnvergence merged commit 5b90b75 into main Oct 6, 2025
7 checks passed
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