+
Skip to content

Conversation

Lucpen
Copy link
Contributor

@Lucpen Lucpen commented Jan 29, 2024

PR checklist

  • Updated pLi config
  • If you've fixed a bug or added code that should be tested, add tests!
  • If you've added a new tool - have you followed the pipeline conventions in the contribution docs
  • If necessary, also make a PR on the nf-core/raredisease branch on the nf-core/test-datasets repository.
  • Make sure your code lints (nf-core lint).
  • Ensure the test suite passes (nextflow run . -profile test,docker --outdir <OUTDIR>).
  • Ensure the test suite passes (nextflow run . -profile test_one_sample,docker --outdir <OUTDIR>).
  • Usage Documentation in docs/usage.md is updated.
  • Output Documentation in docs/output.md is updated.
  • CHANGELOG.md is updated.
  • README.md is updated (including new tool citations and authors/contributors).

This comment was marked as outdated.

@Lucpen Lucpen closed this Jan 29, 2024
Copy link

github-actions bot commented Jan 29, 2024

nf-core lint overall result: Failed ❌

Posted for pipeline commit 85efaa5

+| ✅ 176 tests passed       |+
#| ❔   2 tests were ignored |#
!| ❗   2 tests had warnings |!
-| ❌   3 tests failed       |-

❌ Test failures:

  • files_unchanged - .github/PULL_REQUEST_TEMPLATE.md does not match the template
  • files_unchanged - docs/images/nf-core-raredisease_logo_light.png does not match the template
  • files_unchanged - docs/images/nf-core-raredisease_logo_dark.png does not match the template

❗ Test warnings:

  • pipeline_todos - TODO string in WorkflowRaredisease.groovy: Optionally add in-text citation tools to this list.
  • pipeline_todos - TODO string in awsfulltest.yml: You can customise AWS full pipeline tests as required

❔ Tests ignored:

  • files_exist - File is ignored: conf/modules.config
  • files_unchanged - File ignored due to lint config: .gitignore or .prettierignore or pyproject.toml

✅ Tests passed:

Run details

  • nf-core/tools version 2.12
  • Run at 2024-01-31 12:46:14

@Lucpen Lucpen reopened this Jan 29, 2024
@Lucpen Lucpen changed the base branch from master to dev January 29, 2024 13:11
@Lucpen Lucpen added the ready for review PR is ready for review label Jan 29, 2024
@Lucpen Lucpen requested review from jemten and ramprasadn January 29, 2024 13:28
Copy link
Collaborator

@jemten jemten left a comment

Choose a reason for hiding this comment

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

👍

@jemten
Copy link
Collaborator

jemten commented Jan 31, 2024

merging since the only failures is on nf-core linting. The warnings are unrelated to this PR and will hopefully be fixed with a new version of the nf-core TEMPLATE

@jemten jemten merged commit c1bbb9c into nf-core:dev Jan 31, 2024
@jemten jemten deleted the vep_pli_update branch January 31, 2024 13:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready for review PR is ready for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载