apache / incubator-hugegraph-computer

HugeGraph Computer - A distributed graph processing system for hugegraph (OLAP)
https://hugegraph.apache.org/docs/quickstart/hugegraph-computer/
Apache License 2.0
42 stars 41 forks source link

chore: fix StringEncoding license & add multi license check action #227

Closed simon824 closed 1 year ago

simon824 commented 1 year ago

follow the https://github.com/apache/incubator-hugegraph/issues/1632#issuecomment-1396655141

codecov[bot] commented 1 year ago

Codecov Report

Merging #227 (85ccbd4) into master (3d67f38) will increase coverage by 0.04%. The diff coverage is 40.00%.

@@             Coverage Diff              @@
##             master     #227      +/-   ##
============================================
+ Coverage     85.74%   85.79%   +0.04%     
- Complexity     3232     3233       +1     
============================================
  Files           344      344              
  Lines         12079    12072       -7     
  Branches       1087     1087              
============================================
  Hits          10357    10357              
+ Misses         1197     1191       -6     
+ Partials        525      524       -1     
Impacted Files Coverage Δ
...che/hugegraph/computer/dist/HugeGraphComputer.java 92.45% <ø> (ø)
...e/hugegraph/computer/core/util/StringEncoding.java 17.30% <40.00%> (+2.05%) :arrow_up:
...raph/computer/k8s/operator/OperatorEntrypoint.java 85.71% <0.00%> (-2.53%) :arrow_down:
...k8s/operator/controller/ComputerJobController.java 88.69% <0.00%> (ø)
...h/computer/core/network/session/ClientSession.java 84.42% <0.00%> (+2.45%) :arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

imbajin commented 1 year ago

Another question is: Do we need to bring the reference information of the source code LICENSE in the root to dist/xx/LICENSE for the binary package?

Now we only include it in the source package's LICENSE

WillemJiang commented 1 year ago

Another question is: Do we need to bring the reference information of the source code LICENSE in the dist/xx/LICENSE of the binary package?

Now we only include it in the source package's LICENSE

Yes, we need to include the source code 'LICENSE' information in the binary release 'LICENSE'.