search

nextflow-io / patterns

A curated collection of Nextflow implementation patterns
http://nextflow-io.github.io/patterns/
MIT License
332 stars 71 forks source link

Add nextflow.config and use full github repo path #34

Closed abhi18av closed 2 years ago

abhi18av commented 2 years ago

This PR address the following

1. The warning which is thrown while running the examples since there is no config file.

Pulling nextflow-io/patterns ...
WARN: Cannot read project manifest -- Cause: Remote resource not found: https://api.github.com/repos/nextflow-io/patterns/contents/nextflow.config
 downloaded from https://github.com/nextflow-io/patterns.git

2. Possible confusion regarding the repo name.

Continuing the discussion from https://github.com/nextflow-io/patterns/pull/33#issuecomment-1294946428

I don't follow sorry. Why are they not relevant?

@ewels, sorry it was me being tunnel-visioned at the moment and when I looked at nextflow run pattern/file-name.nf I thought, the users are supposed to clone the repo locally and then run it - which they don't of course and it works fine.

In this PR, I've added the full nextflow-io Github org name so (hopefully) others don't come across the same confusion and it'll avoid the case when people already have the same repo name already clone (via a fork) 

+  >_ nextflow run patterns/conditional-resources.nf
N E X T F L O W  ~  version 22.10.0
Which one do you mean?
nextflow-io/patterns
abhi18av/patterns