-
Notifications
You must be signed in to change notification settings - Fork 1
Remove obsolete scripts - backup runfolder now in automated_scripts, … #42
Conversation
…findfastqs.sh not needed
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.
Reviewed 7 of 7 files at r1, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @RachelDuffin)
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.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @RachelDuffin)
README.md
line 70 at r1 (raw file):
## License Developed by Viapath Genome Informatics
Could we change it to Synnovis please
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.
Reviewable status: 6 of 7 files reviewed, 1 unresolved discussion (waiting on @natashapinto)
README.md
line 70 at r1 (raw file):
Previously, natashapinto (Natasha Pinto) wrote…
Could we change it to Synnovis please
Done
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.
Reviewed 1 of 1 files at r2, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @RachelDuffin)
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.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @RachelDuffin)
…findfastqs.sh not needed
This PR is in conjunction with the automated_scripts v45.0.0 release, as the backup runfolder script now resides in the automated_scripts repo. The find fastqs script is not needed
This change is