PDP-981 : Update copyrightcheck.py for multiline exclude list#39
PDP-981 : Update copyrightcheck.py for multiline exclude list#39brijeshp56 wants to merge 14 commits intomainfrom
Conversation
Updated code to support both single line and multi line exclude list.
There was a problem hiding this comment.
Pull request overview
Updates the copyrightcheck.py configuration parser to support filesexcluded values specified either on a single line (comma-separated) or as a multi-line list, improving flexibility of the script’s exclude configuration.
Changes:
- Add support for multi-line
filesexcluded:blocks in the plain-text config parser. - Refactor config parsing flow to separately handle empty lines vs. comment lines and track an “active” multi-line key.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…ines Update copyrightcheck.py to have exclude file in multiple lines
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Update copyrightcheck.py
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Update copyrightcheck.py
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated 4 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
|
@brijeshp56 I've opened a new pull request, #40, to work on those changes. Once the pull request is ready, I'll request review from you. |
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
|
@brijeshp56 I've opened a new pull request, #41, to work on those changes. Once the pull request is ready, I'll request review from you. |
Agent-Logs-Url: https://github.com/marklogic/pr-workflows/sessions/25a254e4-623b-4097-8ead-02ba997e61eb Co-authored-by: brijeshp56 <203762578+brijeshp56@users.noreply.github.com>
Agent-Logs-Url: https://github.com/marklogic/pr-workflows/sessions/8161bf32-06e4-47c5-b637-c6c6e1ca0037 Co-authored-by: brijeshp56 <203762578+brijeshp56@users.noreply.github.com>
PDP-981 : Remove trailing whitespace in copyrightcheck.py
PDP-981 : Remove trailing whitespace in copyrightcheck.py line 315
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
brijeshp56
requested review from
GAdityaVarma,
Pragathi-28,
SameeraPriyathamTadikonda and
jsvoboda-prgs
Updated code to support both single line and multi line exclude list. Test below scenarios: