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

Update usage.rst #499

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Update usage.rst #499

wants to merge 1 commit into from

Conversation

Leo-GG
Copy link

@Leo-GG Leo-GG commented Nov 12, 2024

Corrected typo hiding the command to run the whole pipeline. Added additional notes about assembly, mapping, classify and annotate rules.

  • I have run pytest .tests/ on a local deployment and the tests passed successfully
  • If this adds a new output file, I have added a check to .tests/targets.txt
  • If this fixes a bug, I have added or modified an appropriate test
  • If this adds or modifies a rule that uses FASTQ files, the input accepts gzipped FASTQ and outputs gzipped FASTQ, or marks uncompressed FASTQ output as temp

If this is for a release:

  • I have updated documentation
  • I have updated the hardcoded version at the top of install.sh to match what this release's version will be
  • I have created a release archive that will be attached to this release (bash dev_scripts/generate_archive.sh)

Corrected typo hiding the command to run the whole pipeline. 
Added additional notes about assembly, mapping, classify and annotate rules.
@Ulthran
Copy link
Contributor

Ulthran commented Nov 13, 2024

Thanks for the contribution @Leo-GG! Looks like something has broken either in snakemake or in one of our container images... I'll have to do some investigating to see what that is. Not something in your PR, this would've been picked up by the next scheduled test run.

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.

2 participants