Skip to content

Commit e665e21

Browse files
committed
On branch edburns/re-enable-pre-commit-hooks
modified: .github/copilot-instructions.md Apply @Copilot review recommendation. > This new section says the pre-commit hook is “automatically enabled” via `copilot-setup-steps.yml`, but earlier in the same document (and in CONTRIBUTING/README) the hook is described as something developers must manually enable with `git config core.hooksPath .githooks`. Please clarify the scope (e.g., auto-enabled only in the Copilot coding agent environment; local development still requires manual enablement) to avoid conflicting guidance, and consider linking explicitly to `.github/workflows/copilot-setup-steps.yml` for precision. Signed-off-by: Ed Burns <edburns@microsoft.com>
1 parent 99d4567 commit e665e21

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

.github/copilot-instructions.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -104,7 +104,7 @@ When porting from .NET:
104104
- 4-space indentation (enforced by Spotless with Eclipse formatter)
105105
- Fluent setter pattern for configuration classes (e.g., `new SessionConfig().setModel("gpt-5").setTools(tools)`)
106106
- Public APIs require Javadoc (enforced by Checkstyle, except `json` and `events` packages)
107-
- Pre-commit hook runs `mvn spotless:check` - enable with: `git config core.hooksPath .githooks`
107+
- Pre-commit hook runs `mvn spotless:check` - Must be manually enabled with: `git config core.hooksPath .githooks`, except in the Copilot coding agent environment. This hook is explicitly enabled in the Copilot coding agent environment. See [copilot-setup-steps.yml](workflows/copilot-setup-steps.yml).
108108

109109
### Handler Pattern
110110

0 commit comments

Comments
 (0)