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 docs for adding new pipelines. #1837

Merged
merged 3 commits into from
Jul 6, 2023

Conversation

ewels
Copy link
Member

@ewels ewels commented Jul 6, 2023

Supersedes #1682

@@ -48,50 +48,64 @@ These topics are specifically discussed in the `#new-pipelines` channel:
## Create a pipeline from the template

You'll start by making a new pipeline locally and working with it on your own GitHub account.
Only when it's ready do we move it to the nf-core GitHub organisation.
Once you have a version of the pipeline that runs, ask the core team to move the repo to the nf-core GitHub organisation for final development, review and release.
Copy link
Member

Choose a reason for hiding this comment

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

Won't this encourage people to write for the first time in #new-pipelines with fully created pipelines? Which is then difficult to review for the 1st release

Copy link
Member Author

Choose a reason for hiding this comment

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

I'll add a new section for this

Copy link
Member Author

Choose a reason for hiding this comment

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

Added, please check 👍🏻

Copy link
Member

Choose a reason for hiding this comment

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

looks good :)

@mribeirodantas
Copy link
Member

@nf-core-bot fix linting

@mribeirodantas mribeirodantas merged commit f83b954 into nf-core:master Jul 6, 2023
2 checks passed
@ewels ewels deleted the adding_pipelines_update branch July 6, 2023 11:11
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.

5 participants