prime-reportstream icon indicating copy to clipboard operation
prime-reportstream copied to clipboard

[SPIKE] Investigate using GitHub Packages for jar file dependencies

Open wcutshall opened this issue 3 months ago • 3 comments

Investigate the possibility of utilizing GitHub Packages as a way to include the hl7v2-fhir-converter library as a ReportStream project dependency in the same way as dependent jar file dependencies are included from Maven Central, etc. The current build uses a Gradle source dependency - which works well with the Gradle command line build but not within the IntelliJ IDE. The purpose of this spike is to see if GitHub Packages can continue to work with the command line build as well as the IntelliJ IDE.

wcutshall avatar Sep 17 '25 17:09 wcutshall

9/25/2025: Bill is investigating, decision will need to be made if we want to move forward with this.

megreed1 avatar Sep 25 '25 19:09 megreed1

TLDR; It is possible to use GitHub Packages for jar file dependencies.

Given the following:

  1. There is no anonymous access to GitHub Packages. One must authenticate with a token in order to read or publish them. (See this thread about NOT allowing anonymous access: https://github.com/orgs/community/discussions/26634)
  2. It will be necessary to publish a version of hl7v2-fhir-converter to GitHub Packages. See this PR.
  3. This PR contains changes for ReportStream.
  4. This same approach could be used for utilizing the prime-fhir-converter library once it is completed.

See these notes on billing: https://docs.github.com/en/billing/concepts/product-billing/github-packages

wcutshall avatar Sep 26 '25 15:09 wcutshall

9/29/2025: decision point- look at user story, PRs. Works as you would expect Maven Central to, need to create indiv personal access token (classic). Bill proved it works with CI, updated one of the actions and it built successfully. Not a lot of work, it's already there in the PRs. Need to decide if this is something we want to do. would be useful for other library we're planning on splitting out

megreed1 avatar Sep 29 '25 19:09 megreed1