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

Generate ukcore snapshots #31

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

declankieran-nhsd
Copy link

@Dunmail Here is a basic version (just linux) of what I mentioned in the other PR.

I couldn't get the snapshot to generate without logging in and running inflate first, so just note there is a fhir login in there. Not ideal, but assuming you have a simplifier account, if you've previously logged in, it seems to find the previous token within the terminal session, so it would be straightforward to handle that appropriately somehow with secret management on your CI server.

I'm using Microsofts linux install script for the dotnet sdk which makes it pretty widely supported on linux distros with different package management. For the firely terminal install, I'm assuming if the command fhir is not available it needs installed. It then overwrites the UKCore profiles in the cache.

Sushi (npm run compile) will then run fine after removing the resource in input/resources, bar the other error with the canonical. I didn't include it here, but there are no errors at all with the other change.

@Dunmail
Copy link
Member

Dunmail commented May 25, 2023

Thanks for this.

@clarotech: This PR relates to #30

Do you know if the proposal would be compatible with the CI build process configured for this repo? My instinct is that the standard FHIR build might lack a) a step where we can run custom scripts b) a means of configuring a simplifier account. I may be wrong though!

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.

3 participants