nf-core / atacseq

ATAC-seq peak-calling and QC analysis pipeline
https://nf-co.re/atacseq
MIT License
187 stars 117 forks source link

Remove empty path components from output paths #372

Closed siddharthab closed 4 months ago

siddharthab commented 4 months ago

GCS does not allow double slashes (//) in paths by default. Other cloud storage platforms also do not have a well-defined behavior for them.

PR checklist

github-actions[bot] commented 4 months ago

This PR is against the master branch :x:


Hi @siddharthab,

It looks like this pull-request is has been made against the siddharthab/atacseq master branch. The master branch on nf-core repositories should always contain code from the latest release. Because of this, PRs to master are only allowed if they come from the siddharthab/atacseq dev branch.

You do not need to close this PR, you can change the target branch to dev by clicking the "Edit" button at the top of this page. Note that even after this, the test will continue to show as failing until you push a new commit.

Thanks again for your contribution!

siddharthab commented 4 months ago

Looks like this was fixed by #355.