You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@paimon.apache.org by "Zouxxyy (via GitHub)" <gi...@apache.org> on 2024/02/21 07:20:26 UTC

[PR] [license] Auto add and unify license header for java & scala file [incubator-paimon]

Zouxxyy opened a new pull request, #2883:
URL: https://github.com/apache/incubator-paimon/pull/2883

   <!-- Please specify the module before the PR name: [core] ... or [flink] ... -->
   
   ### Purpose
   
   Auto add and unify license header for java & scala file, for reviewer, this PR has three parts (I have alreadly divide them):
   
   1.  modify spotless pom
   2. **move other copyright after import** (Because the current correction plan automatically overwrites everything before `package`, **so we need to move other information to avoid being overwritten**)
   3. just auto spotless
   
   ### Tests
   
   <!-- List UT and IT cases to verify this change -->
   
   ### API and Format
   
   <!-- Does this change affect API or storage format -->
   
   ### Documentation
   
   <!-- Does this change introduce a new feature -->
   


-- 
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: issues-unsubscribe@paimon.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] [license] Auto add and unify license header for java & scala file [incubator-paimon]

Posted by "JingsongLi (via GitHub)" <gi...@apache.org>.
JingsongLi merged PR #2883:
URL: https://github.com/apache/incubator-paimon/pull/2883


-- 
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: issues-unsubscribe@paimon.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] [license] Auto add and unify license header for java & scala file [incubator-paimon]

Posted by "Zouxxyy (via GitHub)" <gi...@apache.org>.
Zouxxyy commented on PR #2883:
URL: https://github.com/apache/incubator-paimon/pull/2883#issuecomment-1956240871

   @JingsongLi Scala currently supports automatic generation of license headers, it is very convenient.
   It’s just that java files don’t support this now. Every time I add the license header for java file manually, it’s a bit troublesome.
   
   After this unification,  we can just use `mvn spotless:apply` to automatically add license headers, no more modify


-- 
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: issues-unsubscribe@paimon.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org