Skip to content

Gate publish workflow to upstream org only - #13

Open
matyasselmeci with Copilot wants to merge 2 commits into
mainfrom
copilot/update-publish-workflow-restriction
Open

matyasselmeci with Copilot wants to merge 2 commits into
mainfrom
copilot/update-publish-workflow-restriction

Conversation

Copilot AI commented Sep 15, 2026 •

Copy link
Copy Markdown

This change restricts chart publishing to repositories owned by PelicanPlatform, preventing publish execution in forks while preserving upstream trigger behavior.

  • Scope

    • Updated .github/workflows/publish.yaml only.
    • No trigger/event changes; existing push behavior remains intact for upstream repos.
  • Workflow guard

    • Added a job-level organization check on publish:
      • if: github.repository_owner == 'PelicanPlatform'
    • Result:
      • Upstream (PelicanPlatform/*): job runs as before.
      • Forks (other owners): job is skipped, so no publish step executes.
  • Applied change

    jobs:
      publish:
        name: Package & Push
        if: github.repository_owner == 'PelicanPlatform'
        runs-on: ubuntu-latest

Co-authored-by: matyasselmeci <5246893+matyasselmeci@users.noreply.github.com>
Copilot AI changed the title [WIP] Update publish workflow to restrict execution to upstream organization Gate publish workflow to upstream org only Sep 15, 2026
Copilot AI requested a review from matyasselmeci September 15, 2026 21:47
@matyasselmeci
matyasselmeci marked this pull request as ready for review September 15, 2026 21:47

@matyasselmeci matyasselmeci left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Does what it says on the can

@matyasselmeci
matyasselmeci requested a review from a team September 15, 2026 21:50
@matyasselmeci

Copy link
Copy Markdown
Contributor

(the workflow is disabled on this repo anyway, but that doesn't get inherited by forks, where it just runs and fails)

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