Skip to content

(CAT-2763) Update master references to main - #5

Merged
LukasAud merged 1 commit into
mainfrom
cat_2763_release_puppetlabs_syntax
Aug 20, 2026
Merged

(CAT-2763) Update master references to main#5
LukasAud merged 1 commit into
mainfrom
cat_2763_release_puppetlabs_syntax

Conversation

@gavindidrichsen

Copy link
Copy Markdown
Collaborator

Summary

  • The repo's default branch was renamed mastermain (GitHub API rename, which also flipped the default-branch pointer and required no branch-protection changes since default-protection targets the dynamic ~DEFAULT_BRANCH alias).
  • This updates the remaining hardcoded master references left over from before the rename:
    • README.md — license badge link
    • .github/workflows/ci.yml — push/pull_request trigger branches
    • .github/workflows/mend.yml — pull_request trigger branch

Why

release_prep.yml's target input already defaulted to "main", so a no-argument release-prep dispatch was pointing at a branch that didn't exist until the rename. This PR cleans up the last references to the old branch name.

Test plan

  • Confirmed gh repo view puppetlabs/puppetlabs-syntax --json defaultBranchRef returns main
  • Confirmed the default-protection ruleset still targets ~DEFAULT_BRANCH (dynamic, unaffected by the rename)
  • grep -rn master README.md .github/workflows/ returns no hits after this change

🤖 Generated with Claude Code

Repo default branch was renamed master -> main so release_prep.yml's
default target ("main") resolves correctly. Update the leftover
hardcoded master references in the README badge link and CI/mend
workflow triggers to match.
@gavindidrichsen
gavindidrichsen requested a review from a team as a code owner August 20, 2026 10:34

@LukasAud LukasAud left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

@LukasAud
LukasAud merged commit e6033ce into main Aug 20, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants