-
Notifications
You must be signed in to change notification settings - Fork 10
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
Remove timestamp suffix #97
Conversation
Closes #69 |
|
@apeltzer The fix for the issue is fairly straightforward, it also helps in allowing the module to resume execution from cache. I wasn't able to find a scenario where not having the timestamp suffix would cause any issues, but still I would like to have your input @christopher-mohr to confirm that we can safely remove it |
I think its fine. This was a design decision at some point in the past prior to moving towards nextflow, so I guess its time to get rid of it. Please move forward with the nf-tests :) |
PR checklist
nf-core lint
).nextflow run . -profile test,docker --outdir <OUTDIR>
).nextflow run . -profile debug,test,docker --outdir <OUTDIR>
).docs/usage.md
is updated.docs/output.md
is updated.CHANGELOG.md
is updated.README.md
is updated (including new tool citations and authors/contributors).