Skip to content

[docs] Refresh OB1 PR gate workflow registration#308

Open
Humestone wants to merge 1 commit into
NateBJones-Projects:mainfrom
Humestone:contrib/humestone/ob1-gate-v2
Open

[docs] Refresh OB1 PR gate workflow registration#308
Humestone wants to merge 1 commit into
NateBJones-Projects:mainfrom
Humestone:contrib/humestone/ob1-gate-v2

Conversation

@Humestone
Copy link
Copy Markdown

Summary

  • Rename .github/workflows/ob1-gate.yml to .github/workflows/ob1-gate-v2.yml to force GitHub Actions to register a fresh workflow path.
  • Keep OB1 PR Follow-Ups listening for the declared workflow name, the new path, and the old path during transition.
  • Update stale contributor-facing references that still pointed at the removed ob1-review.yml workflow.

Why

Issue #284 reports that the existing gate workflow has repeatedly created runs with zero jobs and sometimes registers under the workflow path instead of the declared name. A path rename is the smallest first test for workflow registration/cache poisoning while preserving the existing gate logic.

Verification

  • ruby -e 'require "yaml"; Dir[".github/workflows/*.yml"].sort.each { |p| YAML.load_file(p); puts "OK #{p}" }'
  • rg -n "ob1-review.yml|ob1-gate.yml|ob1-gate-v2.yml|OB1 PR Gate" .github CLAUDE.md AGENTS.md CONTRIBUTING.md README.md docs
  • git diff --check

Notes

  • No gate logic changed.
  • actionlint was not available in this local environment; attempted npx --yes actionlint ..., but npm could not determine an executable.
  • Follow-up listener keeps the old path as a transition fallback.

Closes #284 if the new workflow path dispatches jobs normally.

@github-actions github-actions Bot added the documentation Improvements or additions to documentation label May 17, 2026
@github-actions
Copy link
Copy Markdown

Hey @Humestone — welcome to Open Brain Source! 👋

Thanks for submitting your first PR. The automated review will run shortly and check things like metadata, folder structure, and README completeness. If anything needs fixing, the review comment will tell you exactly what.

Once the automated checks pass, a human admin will review for quality and clarity. Expect a response within a few days.

If you have questions, check out CONTRIBUTING.md or open an issue.

@alanshurafa alanshurafa added review: ready-for-maintainer Community reviewer recommends maintainer review area: docs Review area: documentation alan-reviewed Reviewed by Alan Shurafa in Community Reviewer role labels May 20, 2026
@alanshurafa
Copy link
Copy Markdown
Collaborator

Thanks for the contribution. The mechanics are clean: the rename keeps the gate logic 99% identical (only an explanatory comment added), the follow-up listener keeps both the old and new paths so nothing drops mid-transition, and the CLAUDE.md correction is accurate — the file is ob1-gate.yml, not the ob1-review.yml the doc previously referenced.

Workflow-file changes sit outside what I can sign off on as a community reviewer, so I am routing this to the maintainer rather than marking it ready directly. Two things worth the maintainer's attention: whether renaming to force fresh registration is the right fix for the zero-jobs behavior, and whether the old .github/workflows/ob1-gate.yml path should stay in the follow-up listener long-term or be dropped after one clean run.

Recommend maintainer review.

— Alan (community reviewer; non-binding)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

alan-reviewed Reviewed by Alan Shurafa in Community Reviewer role area: docs Review area: documentation documentation Improvements or additions to documentation review: ready-for-maintainer Community reviewer recommends maintainer review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CI] ob1-gate.yml workflow: 0 successful runs in 140 attempts since 2026-03-30 — silent dispatch failure (jobs never start)

2 participants