[PR #6531] [MERGED] Disable workflows in forks #20946

Closed
opened 2026-04-14 21:50:12 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/6531
Author: @jfdoming
Created: 1/3/2026
Status: Merged
Merged: 1/5/2026
Merged by: @jfdoming

Base: masterHead: jfdoming/fork-workflows


📝 Commits (4)

  • 3372181 Add condition to run workflows only for non-forks
  • 66de638 Add condition to skip deployment for forks
  • 852975b Add release notes for CI workflow changes
  • 1a5000f Merge branch 'master' into jfdoming/fork-workflows

📊 Changes

3 files changed (+8 additions, -0 deletions)

View changed files

📝 .github/workflows/docs-release.yml (+1 -0)
📝 .github/workflows/publish-nightly-electron.yml (+1 -0)
upcoming-release-notes/6531.md (+6 -0)

📄 Description

Disable some workflows that require secrets in forks

Summary by CodeRabbit

  • Chores
    • Updated CI/CD workflows to skip deployment and build jobs when running in forked repositories, reducing unnecessary executions and resource use.
  • Documentation
    • Added a release note entry documenting the change to CI behavior.

✏️ Tip: You can customize this high-level summary in your review settings.


🔄 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/actualbudget/actual/pull/6531 **Author:** [@jfdoming](https://github.com/jfdoming) **Created:** 1/3/2026 **Status:** ✅ Merged **Merged:** 1/5/2026 **Merged by:** [@jfdoming](https://github.com/jfdoming) **Base:** `master` ← **Head:** `jfdoming/fork-workflows` --- ### 📝 Commits (4) - [`3372181`](https://github.com/actualbudget/actual/commit/3372181785ba736bf6999ac15e0249493a23aae3) Add condition to run workflows only for non-forks - [`66de638`](https://github.com/actualbudget/actual/commit/66de638a380eeb4901f4ba8e78b34323d7852d9c) Add condition to skip deployment for forks - [`852975b`](https://github.com/actualbudget/actual/commit/852975ba3e8713e8366e8468c81f3b2fb03f9b38) Add release notes for CI workflow changes - [`1a5000f`](https://github.com/actualbudget/actual/commit/1a5000fdbffbf5074e755d453d5b0d3bb344cfd1) Merge branch 'master' into jfdoming/fork-workflows ### 📊 Changes **3 files changed** (+8 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/docs-release.yml` (+1 -0) 📝 `.github/workflows/publish-nightly-electron.yml` (+1 -0) ➕ `upcoming-release-notes/6531.md` (+6 -0) </details> ### 📄 Description Disable some workflows that require secrets in forks <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Chores** * Updated CI/CD workflows to skip deployment and build jobs when running in forked repositories, reducing unnecessary executions and resource use. * **Documentation** * Added a release note entry documenting the change to CI behavior. <sub>✏️ Tip: You can customize this high-level summary in your review settings.</sub> <!-- end of auto-generated comment: release notes by coderabbit.ai --> --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-04-14 21:50:12 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/actual#20946