Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@drpatelh
Copy link
Member

Pseudo-PR for review of full pipeline prior to initial release, as described in https://nf-co.re/developers/adding_pipelines#core-pipeline-review

Do not merge - this PR just provides an interface to look over and comment on all pipeline code. Any suggestions can be applied to the dev branch and this PR will automatically update. Once it has two community reviews, preferably one from someone in @nf-core/core , it can be closed and release 1.0 can go.

Copy link
Member Author

@drpatelh drpatelh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks amazing overall @aunderwo 😍 I have fixed/updated what I can in #38 so just a couple of things left to address that I can see and I think we are good to go.

@antunderwood
Copy link
Collaborator

Hi @drpatelh

I think I resolved your comments. To be pulled in from #39

@drpatelh drpatelh closed this Jun 15, 2021
@drpatelh
Copy link
Member Author

Great work all! Pipeline looks awesome :)

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.

9 participants