Skip to content
Snippets Groups Projects

Update rna-seq.nf

Merged Jonathan Gesell requested to merge 129-Fix_FastQ_Error into develop
All threads resolved!

Please fill in the appropriate checklist below (delete those which are not relevant). These are the most common things requested on pull requests.

Fixed the issue where the script would error out due to not finding precise string. Replaced the string as requested in issue 129.

PR checklist

  • This comment contains a description of changes (with reason)
  • If you've fixed a bug or added code that should be tested, add tests!
  • [ ] Documentation in docs is updated
  • [ ] Replace dag.png with the most recent CI pipeline integrated_pe artifact
  • [ ] Replace software_versions_mqc.yaml with the most recent CI pipeline generateVersions artifact
  • [ ] Replace software_references_mqc.yaml with the most recent CI pipeline generateVersions artifact
  • CHANGELOG.md is updated
  • [ ] README.md is updated
  • [ ] LICENSE.md is updated with new contributors
  • [ ] Docker images moved to production release and changed in pipeline
  • [ ] Docker images used in the CI unit tests match those used in pipeline

/cc @ghenry @venkat.malladi

Edited by Venkat Malladi

Merge request reports

Merge request pipeline #10347 passed

Merge request pipeline passed for 64fa9678

Approval is optional

Merged by Gervaise HenryGervaise Henry 3 years ago (Jul 6, 2021 12:19am UTC)

Merge details

Pipeline #10353 passed

Pipeline passed for bb1e0848 on develop

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply