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.

Sign up for GitHub

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

Jump to bottom

chore(ci): check dependency licenses #460

Merged
merged 2 commits into from
Oct 26, 2022
Merged

chore(ci): check dependency licenses #460

merged 2 commits into from
Oct 26, 2022

Conversation

bednar
Copy link
Contributor

@bednar bednar commented Oct 26, 2022

Proposed Changes

This PR add new step to CircleCI build: check-licenses:

image

The check-licenses uses license-maven-plugin to verify that all dependencies has allowed licence:

influxdb-client-java/pom.xml

Line 243 in a126aeb

<dependencyPolicies>

If there a dependency (akka:2.7.0) which has not allowed license then the build fail with following error:

image

Checklist

  • CHANGELOG.md updated
  • Rebased/mergeable
  • mvn test completes successfully
  • Commit messages are conventional
  • Sign CLA (if not already signed)

@bednar bednar force-pushed the check_dependency_licenses branch from 574110a to c952d0d Compare October 26, 2022 10:25
@bednar bednar marked this pull request as ready for review October 26, 2022 10:32
@bednar bednar requested a review from powersj October 26, 2022 10:32
@bednar bednar merged commit 319c2ef into master Oct 26, 2022
@bednar bednar deleted the check_dependency_licenses branch October 26, 2022 14:50
@bednar bednar added this to the 6.7.0 milestone Oct 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@powersj powersj powersj approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
6.7.0
Development

Successfully merging this pull request may close these issues.
2 participants
@bednar @powersj