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.

Sign up for GitHub

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

Jump to bottom

Effective data requirements reports redundant dependencies for measures #339

Closed
brynrhodes opened this issue Jan 6, 2022 · 1 comment
Closed

Effective data requirements reports redundant dependencies for measures #339

brynrhodes opened this issue Jan 6, 2022 · 1 comment
Labels
bug Something isn't working

Comments

@brynrhodes
Copy link
Member

The refresh measure processing incorrectly sets dependencies (relatedArtifact.where(type = 'depends-on')) in the Measure itself, but those should only be reported in the contained module definition library.

To address this issue, remove this line:
https://github.com/cqframework/cqf-tooling/blob/master/src/main/java/org/opencds/cqf/tooling/measure/MeasureRefreshProcessor.java#L37
@brynrhodes brynrhodes added the bug Something isn't working label Jan 6, 2022
brynrhodes added a commit that referenced this issue Jan 6, 2022
@brynrhodes
#339: Fixed redundant dependencies being reported in measure refresh
fce5564
rob-reynolds added a commit that referenced this issue Feb 22, 2022
@rob-reynolds
Merge pull request #340 from cqframework/fix-339-redundant-dependencies
4c5a9ea 
#339: Fixed redundant dependencies being reported in measure refresh
@brynrhodes
Copy link
Member Author

Included in 1.4.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@brynrhodes