Closed nf-core-bot closed 2 years ago
@drpatelh , not sure what exact I need to do to resolve the following issue
https://github.com/nf-core/gwas/runs/4580839871?check_suite_focus=true#step:6:54
│ [✗] 2 Tests Failed │
├──────────────────────────────────────────────────────────────────────────────┤
│ files_unchanged: assets/nf-core-gwas_logo_light.png does not match the temp… │
│ files_unchanged: docs/images/nf-core-gwas_logo_dark.png does not match the … │
╰──────────────────────────────────────────────────────────────────────────────╯
To keep the code consistent with lots of contributors, we run automated code consistency checks. To fix this CI test, please run:
markdownlint-cli
brew install markdownlint-cli
npm
then install markdownlint-cli
(npm install -g markdownlint-cli
)markdownlint . --fix
markdownlint .
Once you push these changes the test should pass, and you can hide this comment :+1:
We highly recommend setting up markdownlint in your code editor so that this formatting is done automatically on save. Ask about it on Slack for help!
Thanks again for your contribution!
nf-core lint
overall result: Passed :white_check_mark: :warning:Posted for pipeline commit b639443
+| ✅ 140 tests passed |+
!| ❗ 23 tests had warnings |!
Version
2.2
of nf-core/tools has just been released with updates to the nf-core template. This automated pull-request attempts to apply the relevant updates to this pipeline.Please make sure to merge this pull-request as soon as possible, resolving any merge conflicts in the
nf-core-template-merge-2.2
branch (or your own fork, if you prefer). Once complete, make a new minor release of your pipeline.For instructions on how to merge this PR, please see https://nf-co.re/developers/sync.
For more information about this release of nf-core/tools, please see the
v2.2
release page.