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

PR for 1.1.0 release #52

Merged
merged 82 commits into from Mar 29, 2021
Merged

PR for 1.1.0 release #52

merged 82 commits into from Mar 29, 2021

Conversation

apeltzer
Copy link
Member

Many thanks to contributing to nf-core/scrnaseq!

Please fill in the appropriate checklist below (delete whatever is not relevant). These are the most common things requested on pull requests (PRs).

PR checklist

  • This comment contains a description of changes (with reason)
  • If you've fixed a bug or added code that should be tested, add tests!
  • If necessary, also make a PR on the nf-core/scrnaseq branch on the nf-core/test-datasets repo
  • Ensure the test suite passes (nextflow run . -profile test,docker).
  • Make sure your code lints (nf-core lint .).
  • Documentation in docs is updated
  • CHANGELOG.md is updated
  • README.md is updated

Learn more about contributing: https://github.com/nf-core/scrnaseq/tree/master/.github/CONTRIBUTING.md

apeltzer and others added 30 commits December 21, 2019 11:25
Add social preview image
Bump to 1.0.2dev and small bugfixes for docs
Fix thanks to @santiago
@ggabernet ggabernet added this to Ready for review in hackathon-march-2021-pipelines Mar 24, 2021
@apeltzer
Copy link
Member Author

Yeah the failing nf-core lint on release is something that has been fixed in 1.13.3 which is due for release in like 15min? @KevinMenden no?

@ggabernet
Copy link
Member

oh noooo another tools release...

config_profile_description = 'Full test dataset to check pipeline function'

// Input data for full size test
// TODO nf-core: Specify the paths to your full test data ( on nf-core/test-datasets or directly in repositories, e.g. SRA)
Copy link
Member

Choose a reason for hiding this comment

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

It would be great to find real sized test data, but I guess we can also leave it for the DSL2 release if we just wanna get this code out now

docs/usage.md Show resolved Hide resolved
@ggabernet
Copy link
Member

All looks good to me, except there is still the documentation on the parameters that it's not needed any more, and should be removed as it is parsed from the json file now

@apeltzer
Copy link
Member Author

@sk-sahu you want to remove the parameter docs then?

@sk-sahu sk-sahu marked this pull request as draft March 25, 2021 12:04
@sk-sahu sk-sahu marked this pull request as ready for review March 26, 2021 14:08
@sk-sahu sk-sahu requested a review from ggabernet March 26, 2021 15:14
@apeltzer apeltzer merged commit be6daba into master Mar 29, 2021
hackathon-march-2021-pipelines automation moved this from Ready for review to Done Wednesday Mar 29, 2021
@github-actions
Copy link

nf-core lint overall result: Passed ✅ ⚠️

Posted for pipeline commit 1156161

+| ✅ 153 tests passed       |+
!| ❗  21 tests had warnings |!

❗ Test warnings:

  • nextflow_config - Config manifest.version should end in dev: '1.1.0'
  • actions_awsfulltest - .github/workflows/awsfulltest.yml should test full datasets, not -profile test
  • conda_env_yaml - Conda dep outdated: conda-forge::python=3.7.3, 3.9.2 available
  • conda_env_yaml - Conda dep outdated: conda-forge::markdown=3.1.1, 3.3.4 available
  • conda_env_yaml - Conda dep outdated: conda-forge::pymdown-extensions=6.0, 8.1.1 available
  • conda_env_yaml - Conda dep outdated: conda-forge::pygments=2.5.2, 2.8.1 available
  • conda_env_yaml - Conda dep outdated: bioconda::multiqc=1.8, 1.10 available
  • conda_env_yaml - Conda dep outdated: bioconda::bioconductor-isee=1.6.0, 2.2.0 available
  • conda_env_yaml - Conda dep outdated: bioconda::bioconductor-tximeta=1.4.0, 1.8.0 available
  • conda_env_yaml - Conda dep outdated: bioconda::bioconductor-alevinqc=1.2.0, 1.6.0 available
  • conda_env_yaml - Conda dep outdated: bioconda::bioconductor-tximport=1.14.0, 1.18.0 available
  • conda_env_yaml - Conda dep outdated: bioconda::star=2.7.2c, 2.7.8a available
  • conda_env_yaml - Conda dep outdated: samtools=1.10, 1.12 available
  • conda_env_yaml - Conda dep outdated: gffread=0.11.7, 0.12.1 available
  • pipeline_todos - TODO string in README.md: Write a 1-2 sentence summary of what data the pipeline is for and what it does
  • pipeline_todos - TODO string in README.md: Fill in short bullet-pointed list of default steps of pipeline
  • pipeline_todos - TODO string in README.md: If applicable, make list of people who have also contributed
  • pipeline_todos - TODO string in test_full.config: Specify the paths to your full test data ( on nf-core/test-datasets or directly in repositories, e.g. SRA)
  • pipeline_todos - TODO string in test_full.config: Give any required params for the test so that command line flags are not needed
  • pipeline_todos - TODO string in awsfulltest.yml: You can customise AWS full pipeline tests as required
  • pipeline_todos - TODO string in awstest.yml: You can customise CI pipeline run tests as required

✅ Tests passed:

Run details

  • nf-core/tools version 1.13.3
  • Run at 2021-03-26 14:05:59

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

5 participants