dromara / CloudEon

CloudEon uses Kubernetes to install and deploy open-source big data components, enabling the containerized operation of an open-source big data platform. This allows you to reduce your focus on underlying resource management and maintenance.
https://www.cloudeon.top/
Apache License 2.0
429 stars 104 forks source link

fix: Invalid license header of ScriptRunner #67

Closed pan3793 closed 1 year ago

pan3793 commented 1 year ago

What is the purpose of the change

Fix the license header of cloudeon-server/src/main/java/org/dromara/cloudeon/utils/ScriptRunner.java, otherwise, it fails the mvn compile command mvn clean install -DskipTests

[INFO] Replacing main artifact with repackaged archive
[INFO]
[INFO] --- license-maven-plugin:4.2:check (default) @ cloudeon-server ---
[INFO] Checking licenses...
[WARNING] Missing header in: /home/chengpan/Projects/CloudEon/cloudeon-server/src/main/java/org/dromara/cloudeon/utils/ScriptRunner.java
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary for cloudeon v1.0.0:
[INFO]
[INFO] cloudeon ........................................... SUCCESS [ 12.489 s]
[INFO] cloudeon-ui ........................................ SUCCESS [03:47 min]
[INFO] cloudeon-server .................................... FAILURE [02:03 min]
[INFO] cloudeon-assembly .................................. SKIPPED
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  06:10 min
[INFO] Finished at: 2023-06-12T17:21:30+08:00

Brief changelog

Verifying this change

Do I need to test? Has testing been completed? Test method?

Follow this checklist to help us incorporate your contribution quickly and easily. Notice, it would be helpful if you could finish the following 3 checklist before request the community to review your PR.

alifzl commented 1 year ago

use mvn clean install -DskipTests -Dlicense.skip=true

pan3793 commented 1 year ago

@alifzl hi, thanks for the tips, I know this workaround but would like to fix the root cause. And seems there were some issues with the license header clarification, but I believe it is not serious.