Skip to content

fix(ci): skip scheduled workflows on forked repositories - #944

Merged
Teingi merged 1 commit into
oceanbase:mainfrom
wayyoungboy:fix-regression-skip-forks-0528
May 28, 2026
Merged

fix(ci): skip scheduled workflows on forked repositories#944
Teingi merged 1 commit into
oceanbase:mainfrom
wayyoungboy:fix-regression-skip-forks-0528

Conversation

@wayyoungboy

Copy link
Copy Markdown
Member

Summary

  • Forked repos lack required secrets (QWEN_API_KEY, SILICONFLOW_*_API_KEY, PROJECT_ACCESS_TOKEN),
    causing daily CI failures and noisy email notifications for all fork maintainers.
  • Gate schedule and push events in regression.yml on github.repository == 'oceanbase/powermem' so
    fork repos silently skip instead of failing.
  • Apply the same guard to sync-project-milestones.yml.
  • Fix stale project URLs in pyproject.toml and docs/development/overview.md:
    • powermem/powermemoceanbase/powermem (3 URLs)
    • https://powermem.readthedocs.iohttps://www.powermem.ai/docs (2 occurrences, readthedocs
      returns 404)

Changed files

File Change
.github/workflows/regression.yml schedule/push gated on upstream repo; workflow_dispatch and
pull_request_target unchanged
.github/workflows/sync-project-milestones.yml All jobs gated on upstream repo
pyproject.toml Fix [project.urls] — Homepage, Repository, Issues, Documentation
docs/development/overview.md Fix readthedocs link

Motivation

Every fork of this repo receives daily failure emails from the Regression Tests and Sync Project with Milestones workflows. These workflows depend on repo-level secrets that are unavailable in forks, so
they can never succeed. This is the standard approach used by large open-source projects (e.g.,
kubernetes, pytorch) to handle this.

Test plan

  • Verify readthedocs.io returns 404 (confirmed via curl)
  • Verify https://www.powermem.ai/docs returns 200 (confirmed via curl)
  • Verify all ob-labs/* and openclaw/openclaw URLs are still valid (confirmed via

Forked repos lack required secrets (QWEN_API_KEY, SILICONFLOW keys,
PROJECT_ACCESS_TOKEN), causing daily CI failures and noisy email alerts.

- regression.yml: gate schedule/push events on github.repository
- sync-project-milestones.yml: gate all jobs on github.repository
- pyproject.toml: fix project URLs (powermem/powermem → oceanbase/powermem,
  readthedocs.io → powermem.ai/docs)
- docs/development/overview.md: fix stale readthedocs link

@Teingi Teingi left a comment

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.

LGTM

@Teingi
Teingi merged commit ea73d33 into oceanbase:main May 28, 2026
17 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