-
Notifications
You must be signed in to change notification settings - Fork 0
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
Library and Samplesheet Linking for Existing Sequence Runs #889
Conversation
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.
LGTM.
Minor comment, pls unit test some of these later. Okay for now as first pass.
sure, I will add some. And Couple of testing in dev have already conducted, it working fine. It will won't make any breaking changes as it just has insert operation. And I think we can remove it after all history record building finishing. |
Solve: #888
Current Process for processing library linking for existing sequence runs: