Skip to content

Explicit test coverage for Copilot noreply bot email format + README git hooks install instructions#11

Merged
BenjaminMichaelis merged 2 commits into
mainfrom
copilot/fix-coauthor-message
Mar 10, 2026
Merged

Explicit test coverage for Copilot noreply bot email format + README git hooks install instructions#11
BenjaminMichaelis merged 2 commits into
mainfrom
copilot/fix-coauthor-message

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 10, 2026

The Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> trailer format (GitHub Copilot's bot email) had no explicit test coverage, and there were no README instructions for installing the git hooks.

Changes

  • Tests (git-hooks/tests/commit-msg.Tests.ps1): Added explicit test cases for the <NUM+Copilot@users.noreply.github.com> format in both the PowerShell unit tests and bash end-to-end tests. The existing copilot name pattern already matches this — tests lock in the behavior.

    Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
    
  • README: Added a ## Git Hooks section with one-liner install commands for macOS/Linux (bash git-hooks/install.sh) and Windows (powershell -ExecutionPolicy Bypass -File git-hooks/install.ps1).


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

…all instructions

Co-authored-by: BenjaminMichaelis <22186029+BenjaminMichaelis@users.noreply.github.com>
Copilot AI changed the title [WIP] Remove co-author message from git commit Explicit test coverage for Copilot noreply bot email format + README git hooks install instructions Mar 10, 2026
@BenjaminMichaelis BenjaminMichaelis marked this pull request as ready for review March 10, 2026 22:05
@BenjaminMichaelis BenjaminMichaelis merged commit b7b2dcd into main Mar 10, 2026
12 checks passed
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.

2 participants