Skip to content

Log dependencies individually when extracted from IG#669

Merged
cmoesel merged 2 commits intomasterfrom
split-dependency-log
Nov 16, 2020
Merged

Log dependencies individually when extracted from IG#669
cmoesel merged 2 commits intomasterfrom
split-dependency-log

Conversation

@mint-thompson
Copy link
Collaborator

When a configuration is extracted from an ImplementationGuide resource, info messages are logged for the elements of the configuration. The dependencies array may have many elements. To avoid an overly-long message, each element of the array is logged separately.

When a configuration is extracted from an ImplementationGuide resource,
info messages are logged for the elements of the configuration. The
dependencies array may have many elements. To avoid an overly-long
message, each element of the array is logged separately.
Any array element in a configuration that is extracted from an
ImplementationGuide resource is logged the same way. Currently, these
are the dependencies and fhirVersion elements.
Copy link
Contributor

@ngfreiter ngfreiter left a comment

Choose a reason for hiding this comment

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

This looks good to me!

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.

Perfect! And I confirmed that it now plays nicely w/ the IG Publisher, even w/ the project that previously caused issues in the publisher. Thank you, @mint-thompson!

@cmoesel cmoesel merged commit 34c688c into master Nov 16, 2020
@cmoesel cmoesel deleted the split-dependency-log branch November 16, 2020 18:29
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