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

Genmod fixes #247

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from
Draft

Genmod fixes #247

wants to merge 4 commits into from

Conversation

Jakob37
Copy link
Contributor

@Jakob37 Jakob37 commented Oct 29, 2024

Description and reviewer info

What is changed? How does this update improve the pipeline? (For reviewers) How to test it?

Type of change

  • Documentation
  • Patch
  • Minor change
  • Major change

Checklist

  • Self-review of my code
  • Update the CHANGELOG
  • Tag the latest commit (vX.Y.Z format)
  • Log samples used for testing in the Verification_samples_log Excel sheet

Documentation

  • At least one other person has reviewed my changes (not required for trivial changes)

Patch

  • Stub run completes without errors or new warnings
  • At least one other person has reviewed and approved my code (not required for trivial changes)

Major / Minor change

  • Stub run completes without errors or new warnings
  • onco run finishes without any new warnings/errors and the results can
    be loaded into scout
  • wgs single run finishes without any new warnings/errors and the results
    can be loaded into scout
  • wgs trio run finishes without any new warnings/errors and the results
    can be loaded into scout
  • At least one other person has reviewed and approved my code
  • I have made corresponding changes to the documentation (software versions, etc.)

Test/review documentation

Review performed by

  • Alexander
  • Jakob
  • Paul
  • Ryan
  • Viktor

(Add if missing)

Testing performed by

  • Alexander
  • Jakob
  • Paul
  • Ryan
  • Viktor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant