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

Remove unused Cat X licenses #421

Merged
merged 6 commits into from
Sep 22, 2023
Merged

Remove unused Cat X licenses #421

merged 6 commits into from
Sep 22, 2023

Conversation

dacort
Copy link
Contributor

@dacort dacort commented Sep 19, 2023

Testing build for removed license dependencies

What changes were proposed in this pull request?

  • Added exclusions to pom.xml for relevant Cat X licenses

https://issues.apache.org/jira/browse/LIVY-985

How was this patch tested?

  • Unit tests

@codecov-commenter
Copy link

codecov-commenter commented Sep 19, 2023

Codecov Report

Merging #421 (7d6fbea) into branch-0.8 (deffeeb) will increase coverage by 2.56%.
Report is 1 commits behind head on branch-0.8.
The diff coverage is n/a.

@@               Coverage Diff                @@
##             branch-0.8     #421      +/-   ##
================================================
+ Coverage         65.56%   68.13%   +2.56%     
- Complexity          952      975      +23     
================================================
  Files               103      103              
  Lines              6044     6044              
  Branches            911      911              
================================================
+ Hits               3963     4118     +155     
+ Misses             1537     1344     -193     
- Partials            544      582      +38     

see 23 files with indirect coverage changes

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

@dacort dacort changed the title [DO NOT MERGE] Test removing Cat X licenses Remove unused Cat X licenses Sep 22, 2023
@dacort dacort marked this pull request as ready for review September 22, 2023 01:16
Copy link
Contributor

@lmccay lmccay left a comment

Choose a reason for hiding this comment

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

+1

@dacort dacort merged commit 78b5126 into branch-0.8 Sep 22, 2023
6 checks passed
@dacort dacort deleted the fix/test-license branch September 22, 2023 17:49
dacort added a commit that referenced this pull request Oct 12, 2023
* Exclude unnecessary Cat X licenses where possible.
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.

None yet

3 participants