thientd commented on code in PR #21: URL: https://github.com/apache/commons-xml/pull/21#discussion_r3548065556
########## src/test/java/org/apache/commons/xml/XIncludeTest.java: ########## @@ -0,0 +1,420 @@ +/* + * SPDX-FileCopyrightText: 2026 Ta Thien Review Comment: Hi @ppkarwasz, Yes, I agree to remove my copyright line — please go ahead. Tracking authorship through the Git history is perfectly fine by me. Thanks for porting the fix and for checking with me first. I’ve left a suggestion below to drop the SPDX line. The now-empty `*` separator line right after it can go too, so the header matches the canonical ASF form. ```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]
