chore(09): public-repo workflow setup (FLOW-01..05)#1
Merged
Conversation
- Switch CI to ubuntu-latest GitHub-hosted runners (FLOW-01) - Add CODEOWNERS with @DGPisces global rule (FLOW-03) - Add CHANGELOG.md (Keep a Changelog 1.1.0; v1.0.0 entry) (FLOW-05) - Add docs/release/RELEASE-FLOW.md: 8-step manual release recipe (FLOW-05) - Update CONTRIBUTING.md: CI contract for external fork PRs (FLOW-04) - Update install/public-allowlist.md: add CHANGELOG.md + CODEOWNERS Internal source: 4518793 (Phase 9 final commit on internal main)
Logger assignment placed between stdlib and third-party imports was causing 4 E402 violations when ruff scope expanded to full src/ tree. Move log = logging.getLogger(__name__) after all imports.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Phase 9 wires up the public repo so external PRs have the same review + CI bar as internal.
ubuntu-latestGitHub-hosted runners.actions/setup-python@v5used properly.ai-merchantjob guarded with fork-PRif:condition — external forks get lint+unit+playwright (secrets-free); AI scenarios skipped with neutral status.CODEOWNERSat repo root —* @DGPiscesroutes all PRs to the maintainer.CONTRIBUTING.mdupdated with a "CI behavior for external PRs" section explaining the two-tier pipeline.CHANGELOG.mdinitialized (Keep a Changelog 1.1.0; v1.0.0 retroactive entry).docs/release/RELEASE-FLOW.mddocuments the 8-step manual release recipe.What reviewers should check
rufflintssrc/(expanded from explicit file list)ai-merchantjob shows "skipped" status (this is a fork PR, so the condition fires)Internal source
Commits
9a70124..4518793on internal main.Note
Do NOT merge via the GitHub UI until CI passes and maintainer spot-checks the run. Branch protection on
mainis configured in the same Phase 9 work (viagh api).