[PR #136] [MERGED] Fix 'helm-release' CI to disabling chart release for forked repos #226

Closed
opened 2026-10-18 21:07:20 +00:00 by bsakaguchi · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/open-webui/helm-charts/pull/136
Author: @jyje
Created: 1/7/2025
Status: ✅ Merged
Merged: 1/12/2025
Merged by: @james-westbrook

Base: main ← Head: fix/skip-forked-release


📝 Commits (1)

  • 5ae7d88 fix(ci): skip release actions for forked repositories

📊 Changes

1 file changed (+1 additions, -0 deletions)

View changed files

📝 .github/workflows/helm-release.yml (+1 -0)

📄 Description

Hello, folks. I'd like to suggest disabling chart releases for forked repositories by default.

After forking this repository and enabling Actions, I noticed that the main branch's workflow attempted to release charts using unconfigured or void variables, causing the pipeline to fail. This behavior seems common across other repositories with similar setups.

The proposed change is simple: it adds a repository slug check to ensure the workflow only proceeds in the original repository unless explicitly configured otherwise. This helps prevent unnecessary actions and failures in forked repositories. If someone wants to release forked chart, they can comment the if: ... line.

But it’s a simple but significant change, so rejecting this is totally okay. Please check it, and any feedback would be welcome.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/open-webui/helm-charts/pull/136 **Author:** [@jyje](https://github.com/jyje) **Created:** 1/7/2025 **Status:** ✅ Merged **Merged:** 1/12/2025 **Merged by:** [@james-westbrook](https://github.com/james-westbrook) **Base:** `main` ← **Head:** `fix/skip-forked-release` --- ### 📝 Commits (1) - [`5ae7d88`](https://github.com/open-webui/helm-charts/commit/5ae7d88c6de4f6152a09cd60cf6aa2995d77c8ef) fix(ci): skip release actions for forked repositories ### 📊 Changes **1 file changed** (+1 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/helm-release.yml` (+1 -0) </details> ### 📄 Description Hello, folks. I'd like to suggest disabling chart releases for forked repositories by default. After forking this repository and enabling Actions, I noticed that the `main` branch's workflow attempted to release charts using unconfigured or void variables, causing the pipeline to fail. This behavior seems common across other repositories with similar setups. The proposed change is simple: it adds a repository slug check to ensure the workflow only proceeds in the original repository unless explicitly configured otherwise. This helps prevent unnecessary actions and failures in forked repositories. If someone wants to release forked chart, they can comment the `if: ...` line. But it’s a simple but significant change, so rejecting this is totally okay. Please check it, and any feedback would be welcome. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
bsakaguchi 2026-10-18 21:07:20 +00:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
open-webui/helm-charts#226
No description provided.