sanger / limber

A config-driven LIMS built on Sequencescape, primarily for running library preparation pipelines in the laboratory
MIT License
4 stars 8 forks source link

Customer-requested bed number change #2037

Closed KatyTaylor closed 2 weeks ago

KatyTaylor commented 3 weeks ago

Closes #

Changes proposed in this pull request

Requirements from Abby in Slack following end to end lab test:

For bed verification, 2 bed changes:

Need to add a corresponding Int Suite PR.

Instructions for Reviewers

[All PRs] - Confirm PR template filled
[Feature Branches] - Review code
[Production Merges to main]
    - Check story numbers included
    - Check for debug code
    - Check version

codeclimate[bot] commented 3 weeks ago

Code Climate has analyzed commit adebf9a0 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 91.1% (0.0% change).

View more on Code Climate.

codecov[bot] commented 3 weeks ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 77.91%. Comparing base (3ceac2f) to head (adebf9a). Report is 43 commits behind head on develop.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop #2037 +/- ## ======================================== Coverage 77.91% 77.91% ======================================== Files 463 463 Lines 17720 17720 Branches 225 225 ======================================== Hits 13807 13807 Misses 3911 3911 Partials 2 2 ``` | [Flag](https://app.codecov.io/gh/sanger/limber/pull/2037/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=sanger) | Coverage Δ | | |---|---|---| | [javascript](https://app.codecov.io/gh/sanger/limber/pull/2037/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=sanger) | `69.60% <ø> (ø)` | | | [pull_request](https://app.codecov.io/gh/sanger/limber/pull/2037/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=sanger) | `77.91% <ø> (ø)` | | | [push](https://app.codecov.io/gh/sanger/limber/pull/2037/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=sanger) | `77.91% <ø> (ø)` | | | [ruby](https://app.codecov.io/gh/sanger/limber/pull/2037/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=sanger) | `91.16% <ø> (ø)` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=sanger#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.