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

Enable https://github.com/cashapp/licensee #750

Closed
jingtang10 opened this issue Aug 23, 2021 · 3 comments · Fixed by #759
Closed

Enable https://github.com/cashapp/licensee #750

jingtang10 opened this issue Aug 23, 2021 · 3 comments · Fixed by #759
Labels
P1 High priority issue type:process Create or improve processes

Comments

@jingtang10
Copy link
Collaborator

Describe the Issue
Enable the plugin and configure it so it allows only compatible licences in our dependencies.

Would you like to work on the issue?
@kevinmost made a headstart :)

@jingtang10 jingtang10 added the type:process Create or improve processes label Aug 23, 2021
@Tarun-Bhardwaj
Copy link

@kevinmost , could you please provide an ETA for fixing this issue?

@kevinmost
Copy link
Contributor

This change is ready to go, but we are not submitting it until we verify that we can remove a problematic dependency from HAPI FHIR, rather than exempting that dependency in our licensee config.

@Tarun-Bhardwaj Tarun-Bhardwaj added this to To do in FHIR engine library via automation Sep 20, 2021
@Tarun-Bhardwaj Tarun-Bhardwaj moved this from To do to In progress in FHIR engine library Sep 20, 2021
@jingtang10
Copy link
Collaborator Author

With hapifhir/hapi-fhir#3043 merged. I believe we should be able to remove the exception for saxon-he after hapi's next release.

@Tarun-Bhardwaj Tarun-Bhardwaj added the P1 High priority issue label Oct 19, 2021
@jingtang10 jingtang10 linked a pull request Oct 28, 2021 that will close this issue
8 tasks
FHIR engine library automation moved this from In progress to Done Nov 6, 2021
@Tarun-Bhardwaj Tarun-Bhardwaj added Beta and removed Beta labels Nov 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P1 High priority issue type:process Create or improve processes
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants