Skip to content

Replace master references with main after default branch rename#1260

Merged
brendancol merged 1 commit intomainfrom
rename-cleanup-master-to-main
Apr 25, 2026
Merged

Replace master references with main after default branch rename#1260
brendancol merged 1 commit intomainfrom
rename-cleanup-master-to-main

Conversation

@brendancol
Copy link
Copy Markdown
Contributor

Summary

  • Removes master from the CI push-trigger bridge added in Allow CI to trigger on main branch #1255test.yml and benchmarks.yml now trigger only on main.
  • Updates the ASV benchmark comparison baseline from origin/master to origin/main and adjusts the step label.
  • Sets benchmarks/asv.conf.json branches list to ["main"].
  • Updates README.md (license link, Codecov badge) and CONTRIBUTING.md prose.
  • Updates docs/source/index.rst (10 example URLs), docs/source/getting_started/usage.rst, and benchmarks/README.md.
  • Updates .claude/commands/ release, bench, validate, and efficiency-audit scripts.

Context

Second step of the mastermain default branch rename. The first PR (#1255) widened CI triggers to accept both names so there was no coverage gap across the GitHub-side rename. This PR drops the bridge now that main is the default.

A few "master" occurrences are intentionally left alone:

  • actions/checkout@master and actions/setup-python@master in .github/workflows/test.yml — these refer to the master branches of the actions' own repos, not this one. Unpinned @master is a separate hygiene concern worth a follow-up PR to pin to @v4/@v5.
  • master_doc references in docs/source/conf.py and Sphinx templates — this is Sphinx's own config variable name.
  • The ASV default-branch comment in asv.conf.json — documents ASV's upstream behavior, not this project's branch.

Contributor migration

If you have a local clone of this repository, run:

```bash
git branch -m master main
git fetch origin
git branch -u origin/main main
git remote set-head origin -a
git remote prune origin
```

Test plan

  • CI passes on this PR (pull_request trigger fires as before).
  • After merge, a push to `main` triggers both `pytest` and `Benchmarks` workflows.
  • The Codecov badge on `README.md` renders correctly on the `main` branch.
  • Docs rebuild successfully on Read the Docs with the new `/blob/main/` URLs.

Drops master from the CI push-trigger bridge added in #1255 and
updates every remaining reference to the old default branch name:

- test.yml and benchmarks.yml trigger only on main; benchmarks
  compare against origin/main and the step label reads "push to main"
- asv.conf.json branches list now just ["main"]
- README badge and LICENSE link point at main; CONTRIBUTING.md prose
  updated
- docs/source/index.rst and usage.rst example URLs updated
- benchmarks/README.md prose and examples updated
- .claude/commands/ release, bench, validate, and efficiency-audit
  scripts updated

Remaining "master" occurrences in .github/workflows/test.yml
(actions/checkout@master, actions/setup-python@master) refer to the
external actions' own repos and are unrelated to this rename. The
docs/source/conf.py "master_doc" references are Sphinx terminology,
also unrelated. The ASV comment in asv.conf.json documents ASV's
upstream default and stays accurate.
@github-actions github-actions Bot added the performance PR touches performance-sensitive code label Apr 24, 2026
@brendancol brendancol merged commit 71fd943 into main Apr 25, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

performance PR touches performance-sensitive code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant