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

SUSHI 3.0.0 + Relevant Features #222

Merged
merged 7 commits into from
Jun 8, 2023
Merged

SUSHI 3.0.0 + Relevant Features #222

merged 7 commits into from
Jun 8, 2023

Conversation

jafeltra
Copy link
Collaborator

@jafeltra jafeltra commented Jun 7, 2023

PR to include SUSHI 3.0.0 and the latest FPL release, along with the features we've implemented to support those releases.

All these changes have been reviewed except for the last two commits (bd2c4d0 and bc7b0b3), which actually update the package versions.

jafeltra and others added 7 commits May 1, 2023 11:25
* Replace loadDependency and loadFromPath functions with FPL functions

* Add FPL package and repository field

* Use mergeDependency function and update tests

* Use FPL FHIRDefinitions class with a simpler fishForMetadata implementation

* Update to reflect latest changes in SUSHI

* Add log function to pass in to FPL functions
* Create inline instances with numeric ids

* Use node 16, 18 for automated testing

* Update test for numeric id
* Handle numeric element names

Elements are allowed to have numeric names. When flattening an object,
array indices get put into square brackets, but numeric element names do
not. This keeps the paths intact when checking types and writing
assignment or caret value rules.

* fix parameter name

* Use node 16, 18 for automated testing

* Add test for processing Instance with numeric paths

Add test for extracting caret rules on an array element containing an
empty object.
Update tool versions.
Improve implementation of getPathValuePairs.
@jafeltra jafeltra requested review from cmoesel and mint-thompson June 7, 2023 20:11
Copy link
Collaborator

@mint-thompson mint-thompson left a comment

Choose a reason for hiding this comment

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

hooray for sushi v3.0.0!

Copy link
Member

@cmoesel cmoesel left a comment

Choose a reason for hiding this comment

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

Hurray indeed!

@cmoesel cmoesel merged commit b79c046 into master Jun 8, 2023
@cmoesel cmoesel deleted the next-v3 branch June 8, 2023 16:14
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