Skip to content

chore(quantecon): switch to merge-to-main development model#23

Merged
mmcky merged 2 commits into
mainfrom
chore/switch-to-merge-to-main-model
May 14, 2026
Merged

chore(quantecon): switch to merge-to-main development model#23
mmcky merged 2 commits into
mainfrom
chore/switch-to-merge-to-main-model

Conversation

@mmcky
Copy link
Copy Markdown

@mmcky mmcky commented May 14, 2026

Summary

  • Remove the integration-branch tooling (quantecon/build.sh, quantecon/features.txt)
  • Rewrite quantecon/README.md for the new workflow: features squash-merge into main; feature branches are preserved post-merge as upstream PR artifacts
  • Projects install from main instead of the discontinued quantecon integration branch

Why

The previous model kept feature branches unmerged and built a throwaway quantecon integration branch via build.sh. That worked for one feature, but doesn't scale: stacked unmerged branches drift, rebase pain compounds, and projects have to track a custom tip nobody else uses.

A merged branch is still upstreamable. Merging feature/<name> into our main doesn't alter the branch itself — when upstream is ready, we push the preserved branch and open a PR against jupyter-book/mystmd:main. Same commits, same diff, same review experience.

Net effect: ~200 lines of bash gone, simpler dev process, no integration-branch failure modes, upstream PR readiness preserved.

Sequencing

Land this PR first, then squash-merge the open feature PRs into main:

  1. feat(ipynb): CommonMark ipynb export + image attachment embedding #16feature/myst-to-ipynb
  2. Book-style numbering: format, label, section-tagged TOC, auto-prefix #22feature/book-numbering

Both feature branches stay alive after merging for the eventual upstream PRs.

Test plan

  • CI passes (lint, tests, docs build) — no code changes, but the workflow runs on every PR
  • README renders correctly on GitHub
  • Confirm quantecon/build.sh and quantecon/features.txt are gone after merge
  • After merge: git clone --branch main + bun install && bun run build still works (no code changed, this should be a no-op)

🤖 Generated with Claude Code

Remove the integration-branch tooling (build.sh, features.txt) and
rewrite the maintenance guide. Features now squash-merge directly into
main; feature branches are preserved post-merge as upstream PR artifacts.

This removes ~200 lines of bash and a class of "the integration branch
is stale" failure modes. Projects install from main instead of the
discontinued `quantecon` branch.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings May 14, 2026 04:46
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Documentation/tooling-only change that retires the throwaway quantecon integration branch in favor of squash-merging feature branches directly into this fork's main, while keeping the original feature branches alive as upstream PR artifacts.

Changes:

  • Delete quantecon/build.sh and quantecon/features.txt (no longer needed under the merge-to-main model).
  • Rewrite quantecon/README.md to document the new workflow: branch from upstream/main, PR into QuantEcon/mystmd:main, squash-merge, preserve branch for upstream PR.
  • Update install instructions in the README to clone --branch main instead of --branch quantecon.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
quantecon/README.md Rewrites the maintenance guide to describe the squash-merge-to-main workflow and updates GitHub Actions install snippet to use main.
quantecon/features.txt Removed; feature tracking is now done via live feature/* branches.
quantecon/build.sh Removed; integration branch is no longer built.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

…t README)

Adds quantecon/.gitignore so PLAN docs, demo books, and other local
experiments can live alongside the maintenance guide without risk of
accidental commits. Only README.md and .gitignore itself are tracked.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@mmcky mmcky merged commit 73b2946 into main May 14, 2026
5 checks passed
@mmcky mmcky deleted the chore/switch-to-merge-to-main-model branch May 14, 2026 04:55
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