Skip to content

chore: fix branch matching in ignoreBranches #281

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

famouswizard
Copy link

@famouswizard famouswizard commented Apr 10, 2025

dev/feature-123 wasn’t ignored due to exact match regex.
updated the pattern to handle nested branches.


PR-Codex overview

This PR updates the branch name validation logic in the checkBranchNames.js script to ensure that the ignoreBranches regex correctly handles branches named main and dev, allowing for sub-branches.

Detailed summary

  • Modified the ignoreBranches regex from /^(main|dev)$/ to /^(main|dev)(\/|$)/ to accommodate sub-branches under main and dev.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

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.

1 participant