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

Make modality required for all extractions #2045

Merged

jas88 merged 29 commits into main from change/require-modality

Dec 11, 2024

Merged

use var instead of type

Codecov / codecov/patch succeeded Dec 11, 2024 in 0s

83.82% of diff hit (target 65.67%)

83.82% of diff hit (target 65.67%)

src/SmiServices/Microservices/CohortExtractor/CohortExtractorHost.cs#L123

Added line #L123 was not covered by tests

src/SmiServices/Microservices/CohortExtractor/CohortExtractorHost.cs#L127

Added line #L127 was not covered by tests

src/SmiServices/Microservices/CohortExtractor/RequestFulfillers/ExtractionRequestFulfillerFactory.cs#L14

Added line #L14 was not covered by tests