Skip to content

Conversation

@colinlienard
Copy link
Member

@colinlienard colinlienard commented Jan 8, 2026

📝 What does this change do? Why?

Add claude prefix

@colinlienard colinlienard self-assigned this Jan 8, 2026
@github-actions github-actions bot added the feat New Feature label Jan 8, 2026
@colinlienard colinlienard marked this pull request as ready for review January 8, 2026 10:02
Copilot AI review requested due to automatic review settings January 8, 2026 10:02
Copy link

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

This PR adds support for the "claude" branch prefix to the branch name validation function, allowing branches prefixed with "claude/" to be considered valid. This follows the same pattern as existing AI/automation tool prefixes like "copilot", "renovate", and "dependabot".

  • Added regex pattern for "claude" prefix validation in the branch name validator
  • Added test case to verify "claude/foo" is a valid branch name

Reviewed changes

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

File Description
lib/branch.js Added regex pattern to validate branch names with "claude/" prefix, consistent with other automation tool patterns
lib/branch.test.js Added test case "claude/foo" to verify the new prefix is accepted as valid

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

Copy link
Contributor

@callms callms left a comment

Choose a reason for hiding this comment

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

LGTM

@pierresisson pierresisson merged commit 82127cf into master Jan 8, 2026
15 checks passed
@pierresisson pierresisson deleted the feat/fttech-5488-ajout-du-prefixe-de-branche-claude-pour-la-gha-compliance branch January 8, 2026 12:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feat New Feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants