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

Record number of skipped executions during aggregation #486

Merged
merged 11 commits into from
Mar 8, 2024

Use 1.16.0-alpha.2 webservice version

6696cc6
Select commit
Loading
Failed to load commit list.
Merged

Record number of skipped executions during aggregation #486

Use 1.16.0-alpha.2 webservice version
6696cc6
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Mar 7, 2024 in 0s

87.85% of diff hit (target 56.41%)

View this Pull Request on Codecov

87.85% of diff hit (target 56.41%)

Annotations

Check warning on line 29 in metricsaggregator/src/main/java/io/dockstore/metricsaggregator/helper/ExecutionTimeAggregator.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

metricsaggregator/src/main/java/io/dockstore/metricsaggregator/helper/ExecutionTimeAggregator.java#L29

Added line #L29 was not covered by tests

Check warning on line 34 in metricsaggregator/src/main/java/io/dockstore/metricsaggregator/helper/ValidationStatusAggregator.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

metricsaggregator/src/main/java/io/dockstore/metricsaggregator/helper/ValidationStatusAggregator.java#L34

Added line #L34 was not covered by tests

Check warning on line 39 in metricsaggregator/src/main/java/io/dockstore/metricsaggregator/helper/ValidationStatusAggregator.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

metricsaggregator/src/main/java/io/dockstore/metricsaggregator/helper/ValidationStatusAggregator.java#L39

Added line #L39 was not covered by tests