-
Notifications
You must be signed in to change notification settings - Fork 235
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
Modify MappingModelCoverageAnalysis to return PMCD for the model coverage #2152
Conversation
Test Results 687 files + 687 687 suites +687 1h 1m 10s ⏱️ + 1h 1m 10s Results for commit 17c507c. ± Comparison against base commit fb42a4c. ♻️ This comment has been updated with latest results. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You might want to rebase and update snapshot version number
...h/legend-engine-xt-analytics-search-pure/src/main/resources/core_analytics_search/trans.pure
Show resolved
Hide resolved
...alytics-mapping-pure/src/main/resources/core_analytics_mapping/modelCoverage/serializer.pure
Outdated
Show resolved
Hide resolved
...-analytics-mapping-pure/src/main/resources/core_analytics_mapping/modelCoverage/utility.pure
Show resolved
Hide resolved
024cf48
to
5a36920
Compare
461351e
to
47ef1aa
Compare
74a7a3e
to
da0a0ec
Compare
da0a0ec
to
1aeef1a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approving change in PureIDELight file
1aeef1a
to
e901cab
Compare
What type of PR is this?
Improvement
What does this PR do / why is it needed ?
Which issue(s) this PR fixes:
Related to finos/legend-studio#2346
Other notes for reviewers:
Does this PR introduce a user-facing change?