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

[O2B-1365] Disable GAQ summary if not all detectors have full coverage #1784

Draft
wants to merge 9 commits into
base: main
Choose a base branch
from

gaq summary disabled when missing coverage

ec92f8d
Select commit
Loading
Failed to load commit list.
Draft

[O2B-1365] Disable GAQ summary if not all detectors have full coverage #1784

gaq summary disabled when missing coverage
ec92f8d
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Oct 14, 2024 in 1s

60.00% of diff hit (target 43.90%)

View this Pull Request on Codecov

60.00% of diff hit (target 43.90%)

Annotations

Check warning on line 167 in lib/database/repositories/QcFlagRepository.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

lib/database/repositories/QcFlagRepository.js#L167

Added line #L167 was not covered by tests

Check warning on line 288 in lib/database/repositories/QcFlagRepository.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

lib/database/repositories/QcFlagRepository.js#L288

Added line #L288 was not covered by tests

Check warning on line 170 in lib/public/views/Runs/RunPerDataPass/RunsPerDataPassOverviewPage.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

lib/public/views/Runs/RunPerDataPass/RunsPerDataPassOverviewPage.js#L170

Added line #L170 was not covered by tests

Check warning on line 708 in lib/server/services/qualityControlFlag/QcFlagService.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

lib/server/services/qualityControlFlag/QcFlagService.js#L708

Added line #L708 was not covered by tests