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

[#1044] chore: Add LICENSE-binary #1054

Merged
merged 10 commits into from
Aug 14, 2023
Merged

Conversation

jerqi
Copy link
Contributor

@jerqi jerqi commented Jul 30, 2023

What changes were proposed in this pull request?

Add Notice-binary

Why are the changes needed?

Fix: #1044
If we want to release binary package, we should have LICENSE-binary. This is very important. The IPMC will check these licenses.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

No need.

@codecov-commenter
Copy link

codecov-commenter commented Jul 30, 2023

Codecov Report

Merging #1054 (56d86d0) into master (1c99972) will increase coverage by 1.18%.
Report is 1 commits behind head on master.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #1054      +/-   ##
============================================
+ Coverage     54.06%   55.24%   +1.18%     
  Complexity     2541     2541              
============================================
  Files           386      366      -20     
  Lines         21879    19519    -2360     
  Branches       1812     1812              
============================================
- Hits          11828    10784    -1044     
+ Misses         9355     8108    -1247     
+ Partials        696      627      -69     

see 23 files with indirect coverage changes

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

LICENSE-binary Outdated Show resolved Hide resolved
LICENSE-binary Outdated Show resolved Hide resolved
LICENSE-binary Outdated Show resolved Hide resolved
LICENSE-binary Outdated Show resolved Hide resolved
LICENSE-binary Outdated
org.apache.httpcomponents:httpclient
net.jpountz.lz4:lz4
com.google.guava:guava
com.google.guava::listenablefuture
Copy link
Contributor

Choose a reason for hiding this comment

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

::?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed.

@jerqi
Copy link
Contributor Author

jerqi commented Aug 14, 2023

@LuciferYang Could we merge this pr first?

@LuciferYang LuciferYang merged commit 72b8698 into apache:master Aug 14, 2023
30 checks passed
@LuciferYang
Copy link
Contributor

Merged into master. Thanks @jerqi

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Improvement] Add LICENSE-binary
3 participants