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.

Sign up for GitHub

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

Jump to bottom

Small mistake in usage.md #327

Closed
heseber opened this issue Jan 19, 2021 · 2 comments
Closed

Small mistake in usage.md #327

heseber opened this issue Jan 19, 2021 · 2 comments

Comments

@heseber
Copy link

heseber commented Jan 19, 2021

The paragraph talks aboutstar, the code example uses VEP.

image
@heseber
Copy link
Author

heseber commented Jan 20, 2021

By the way, it would probably be worthwhile using withLabel here to catch both VEP and VEPmerge.

@maxulysse
Copy link
Member

The paragraph talks aboutstar, the code example uses VEP.

Very good catch

By the way, it would probably be worthwhile using withLabel here to catch both VEP and VEPmerge.

Good idea

I'll add all that to my current PR

maxulysse added a commit to maxulysse/nf-core_sarek that referenced this issue Jan 20, 2021
@maxulysse
fix: update docs to close nf-core#327
2b62a1a
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@maxulysse @heseber