Copilot commented on code in PR #12071:
URL: https://github.com/apache/cloudstack/pull/12071#discussion_r2524775740
##########
.pre-commit-config.yaml:
##########
@@ -62,6 +62,15 @@ repos:
- .github/workflows/license-templates/LICENSE.txt
- --fuzzy-match-generates-todo
exclude:
^(CHANGES|ISSUE_TEMPLATE|PULL_REQUEST_TEMPLATE)\.md$|^ui/docs/(full|smoke)-test-plan\.template\.md$
+ - id: insert-license
+ name: add license for all SQL files
+ files: \.sql$
+ args:
+ - --comment-style
+ - '|--|'
Review Comment:
The comment-style format appears to be incorrect. For the
Lucas-C/pre-commit-hooks insert-license hook, the comment-style format should
be `'|<prefix>|<suffix>|'`.
For SQL single-line comments using `--`, the correct format should be
`'|--||'` (with an empty suffix) rather than `'|--|'`. The current format
`'|--|'` may not work correctly with the pre-commit hook.
Compare with the Markdown configuration on line 60: `'<!--|| -->'` where
`<!--` is the prefix and ` -->` is the suffix.
```suggestion
- '|--||'
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
