docs: add fork contribution workflow to CONTRIBUTING.md - #219
Merged
bradygaster merged 2 commits intoMar 7, 2026
Merged
Conversation
) Closes bradygaster#218 Five gaps identified during PR bradygaster#217 contribution from a fork: - Fork-first setup instructions in Getting Started - PR process now specifies --base dev and explicit --head - Changeset added as a required step in the PR checklist - Branch Strategy: bradygaster/dev clarified as PR target for all contributions - New 'Keeping Your Fork in Sync' section with rebase instructions Also captures fork procedure in Kobayashi's charter (permanent procedure) and history (lessons from today's session).
Rewrote 'Fork Contribution Procedure' entry to contain only reusable procedural knowledge — removed PR numbers, dates, and 'what went wrong' narrative. Kept all actionable commands.
bradygaster
added a commit
that referenced
this pull request
Mar 7, 2026
Session: 2026-03-07T01-13-00Z-phase2-complete Requested by: Brady Changes: - Merged 3 inbox decisions (Kobayashi PR merges, Keaton community PRs, Fenster template fix) - Logged orchestration results for all 3 agents (Kobayashi, Keaton, Hockney) - Logged Phase 2 session completion - Appended Phase 2 results to agent history files (cross-agent updates) - Deduplicated: No duplicate decisions found; all 3 inbox items were new. Merges completed: - PR #232: Scribe runtime state fix (86598f4) - PR #212: Version stamp preservation (0fedcce) - PR #230: CLI wire-up squad link + squad init --mode remote (6d0bd56) - PR #217: TUI /init no-args flow fix (20970f9) - PR #219: Fork contribution workflow docs (157b8c0) Test results: 130/134 test files passing. 12 pre-existing failures in consult, REPL, status, acceptance modes. Core CLI solid.
jongio
pushed a commit
to jongio/squad
that referenced
this pull request
Mar 9, 2026
* feat(cli): add deprecation notice to GitHub-native distribution (bradygaster#219) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * test: add edge-case coverage for SDK distribution resolution (bradygaster#180) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
jongio
pushed a commit
to jongio/squad
that referenced
this pull request
Mar 9, 2026
…ster#219) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
jongio
pushed a commit
to jongio/squad
that referenced
this pull request
Mar 9, 2026
* feat(cli): add deprecation notice to GitHub-native distribution (bradygaster#219) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * chore: bump to v0.6.0 for production release (bradygaster#216) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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
2 participants
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.
Closes #218
Summary
Five gaps in CONTRIBUTING.md identified during PR #217 (contributing from a fork). This PR fixes them all.
Changes
Getting Started — Fork-First Workflow
Replaced the bare
git cloneinstruction with a full 4-step fork setup: fork on GitHub → clone fork → add upstream remote → fetch upstream dev.New: Keeping Your Fork in Sync
Added a subsection explaining how to rebase on
upstream/devbefore opening or updating a PR.Pull Request Process
gh pr createwith explicit base/head:gh pr create --base dev --repo bradygaster/squad --head {yourusername}:branchBranch Strategy
Updated
bradygaster/devdescription from "Brady's integration branch (team usage)" to "Integration branch. All PRs from forks must target this branch, notmain".Kobayashi charter + history (team-internal)