Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: support check license header with RAT & fill the gaps #1988

Merged
merged 14 commits into from
Oct 26, 2022

Conversation

z7658329
Copy link
Member

support check license header with RAT

@codecov
Copy link

codecov bot commented Oct 25, 2022

Codecov Report

Merging #1988 (8c22c5a) into master (9b5950e) will increase coverage by 0.00%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             master    #1988    +/-   ##
==========================================
  Coverage     70.54%   70.54%            
+ Complexity      978      724   -254     
==========================================
  Files           454      454            
  Lines         39055    39055            
  Branches       5557     5557            
==========================================
+ Hits          27550    27551     +1     
+ Misses         8806     8804     -2     
- Partials       2699     2700     +1     
Impacted Files Coverage Δ
...ava/com/baidu/hugegraph/api/gremlin/CypherAPI.java 0.00% <ø> (ø)
...m/baidu/hugegraph/api/gremlin/GremlinQueryAPI.java 63.88% <ø> (ø)
...ain/java/com/baidu/hugegraph/define/Checkable.java 100.00% <ø> (ø)
.../java/com/baidu/hugegraph/backend/store/Shard.java 62.50% <ø> (ø)
...java/com/baidu/hugegraph/job/schema/SchemaJob.java 52.94% <ø> (ø)
...va/com/baidu/hugegraph/type/define/Directions.java 86.11% <ø> (ø)
...hugegraph/backend/store/raft/rpc/RaftRequests.java 19.01% <0.00%> (ø)
...ugegraph/backend/store/mysql/ResultSetWrapper.java 62.50% <ø> (ø)
...hugegraph/backend/store/raft/rpc/RpcForwarder.java 69.49% <0.00%> (-1.70%) ⬇️
... and 2 more

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

Copy link
Contributor

@javeme javeme left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for your contribution~

imbajin
imbajin previously approved these changes Oct 26, 2022
Dockerfile Show resolved Hide resolved
@imbajin imbajin changed the title support check license header with RAT chore: support check license header with RAT & fill the gaps Oct 26, 2022
@imbajin imbajin merged commit 8abf3d5 into apache:master Oct 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants