Skip to content

Sys-8303 co-pilot autofix#37

Merged
mmaharjan-ccdc merged 3 commits into
mainfrom
SYS-8303-co-pilot-autofix
May 7, 2026
Merged

Sys-8303 co-pilot autofix#37
mmaharjan-ccdc merged 3 commits into
mainfrom
SYS-8303-co-pilot-autofix

Conversation

@mmaharjan-ccdc
Copy link
Copy Markdown
Contributor

No description provided.

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

Updates the commit-msg hook logic to allow GitHub Copilot Autofix commits (identified via a specific Co-authored-by: line) to bypass the Jira ID requirement, and documents this behavior in the README.

Changes:

  • Allow commits to pass the Jira ID check when a Copilot Autofix co-author line is present in the commit message.
  • Add a unit test covering the Copilot Autofix commit-message scenario.
  • Update README to reflect the new Jira ID exception.

Reviewed changes

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

File Description
README.md Documents the new exception for Copilot Autofix co-author lines when enforcing Jira IDs.
main/githooks.py Implements Copilot Autofix co-author detection in check_commit_msg and adds a corresponding test case.

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

Comment thread README.md Outdated
ID (unless marked with NO_JIRA), or if the size of new or modiffied files
exceeds a threshold.
ID (unless marked with NO_JIRA or a Copilot Autofix co-author line), or if the
size of new or modiffied files exceeds a threshold.
Comment thread main/githooks.py Outdated
# No jira marker in commit message
NO_JIRA_MARKER = 'NO_JIRA'
# Copilot Autofix co-author in commit message description
COPILOT_AUTOFIX = re.compile(
@mmaharjan-ccdc mmaharjan-ccdc changed the title Sys 8303 co pilot autofix Sys-8303 co-pilot autofix May 7, 2026
@mmaharjan-ccdc mmaharjan-ccdc merged commit 6217d92 into main May 7, 2026
6 checks passed
@mmaharjan-ccdc mmaharjan-ccdc deleted the SYS-8303-co-pilot-autofix branch May 7, 2026 13:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants