Skip to content

Skip all CI on forks - #329

Open
samaloney wants to merge 6 commits into
sunpy:mainfrom
samaloney:feat-limited-fork-ci
Open

samaloney wants to merge 6 commits into
sunpy:mainfrom
samaloney:feat-limited-fork-ci

Conversation

@samaloney

@samaloney samaloney commented Sep 21, 2026 •

Copy link
Copy Markdown
Member

PR Description

Don't run sub_package_update or label_sync on forks at all, main CI don't run scheduled jobs on forks.

AI Assistance Disclosure

AI tools were used for:

  • Code generation (e.g., when writing an implementation or fixing a bug)
  • Test/benchmark generation
  • Documentation (including examples)
  • Research and understanding
  • [x ] No AI tools were used

Regardless of AI use, the human contributor remains fully responsible for correctness, design choices, licensing compatibility, and long-term maintainability.

@samaloney samaloney changed the title Skip labels sync and sub package update on forks Skip all CI on forks Sep 21, 2026
@samaloney

Copy link
Copy Markdown
Member Author

So this is pretty blunt essentially disable all CI on forks wholesale with no override. I guess if test something you might actually want the CI to run in your fork

Comment thread {{ cookiecutter.package_name }}/.github/workflows/ci.yml Outdated

jobs:
core:
if: {{ "${{ !github.event.repository.fork || github.event_name != 'schedule' }}" }}

@samaloney samaloney Sep 22, 2026 •

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

run if not a fork or not schedule so always run on upstream don't run if on a fork and scheduled event ...
hmm maybe this is clearer !(github.event.repository.fork && github.event_name == 'schedule')

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Do people use the CI on their forks at all?

@samaloney
samaloney marked this pull request as ready for review September 22, 2026 11:40

This branch has not been deployed

No deployments
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