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.

Sign up for GitHub

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

Jump to bottom

Feat/nf tests #309

Open
wants to merge 17 commits into
base: dev
Choose a base branch
from
Open

Feat/nf tests #309

wants to merge 17 commits into from

Conversation

sateeshperi
Copy link

✨ nf-tests

@github-actions

This comment was marked as outdated.

Sign in to view
@sateeshperi sateeshperi self-assigned this Jun 19, 2023
@sateeshperi sateeshperi marked this pull request as draft June 19, 2023 20:38
@sateeshperi sateeshperi added the enhancement Improvement for existing functionality label Jun 19, 2023
@github-actions

This comment was marked as outdated.

Sign in to view
@github-actions
Copy link

github-actions bot commented Jun 19, 2023 •
edited
Loading

nf-core lint overall result: Failed ❌

Posted for pipeline commit ec28e57

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

❌ Test failures:

  • files_unchanged - .github/CONTRIBUTING.md does not match the template
  • files_unchanged - .github/PULL_REQUEST_TEMPLATE.md does not match the template
  • files_unchanged - .github/workflows/linting.yml does not match the template
  • files_unchanged - lib/NfcoreTemplate.groovy does not match the template
  • multiqc_config - 'assets/multiqc_config.yml' does not contain a matching 'report_comment'.
    The expected comment is:
    This report has been generated by the <a href="https://github.com/nf-core/taxprofiler/tree/dev" target="_blank">nf-core/taxprofiler</a> analysis pipeline. For information about how to interpret these results, please see the <a href="https://nf-co.re/taxprofiler/dev/docs/output" target="_blank">documentation</a>.
    The current comment is:
    This report has been generated by the <a href="https://github.com/nf-core/taxprofiler/releases/tag/dev" target="_blank">nf-core/taxprofiler</a> analysis pipeline. For information about how to interpret these results, please see the <a href="https://nf-co.re/taxprofiler/dev/docs/output" target="_blank">documentation</a>.

❗ Test warnings:

  • pipeline_todos - TODO string in main.nf: Remove this line if you don't need a FASTA file
  • pipeline_todos - TODO string in methods_description_template.yml: #Update the HTML below to your preferred methods description, e.g. add publication citation for this pipeline

❔ Tests ignored:

✅ Tests passed:

Run details

  • nf-core/tools version 2.11.1
  • Run at 2024-01-12 05:41:31

sateeshperi
sateeshperi commented Jun 22, 2023
View reviewed changes
.github/workflows/ci.yml Outdated Show resolved Hide resolved
@sateeshperi sateeshperi marked this pull request as ready for review June 26, 2023 20:51
@jfy133 jfy133 mentioned this pull request Jun 29, 2023
Open
@jfy133 jfy133 changed the base branch from dev to bouncy-basenji November 13, 2023 11:02
@jfy133
Copy link
Member

jfy133 commented Nov 13, 2023

@nf-core-bot fix linting

Base automatically changed from bouncy-basenji to dev September 11, 2024 05:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@sateeshperi sateeshperi

Labels
enhancement Improvement for existing functionality
Projects
nf-test Migration
Status: In Progress
Hackathon: November 2023
Status: In progress
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@sateeshperi @jfy133 @nf-core-bot