Skip to content

Patch aws full test config#101

Closed
vagkaratzas wants to merge 3 commits into
mainfrom
patch-aws-test-config
Closed

Patch aws full test config#101
vagkaratzas wants to merge 3 commits into
mainfrom
patch-aws-test-config

Conversation

@vagkaratzas
Copy link
Copy Markdown
Collaborator

@vagkaratzas vagkaratzas commented May 8, 2026

Bug fix to allow AWS full test to run on Seqera Platfrom properly

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 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/proteinannotator branch on the nf-core/test-datasets repository.
  • Make sure your code lints (nf-core pipelines lint).
  • Ensure the test suite passes (e.g. nf-test test */local --profile=~test,docker for all new local tests).
  • Check for unexpected warnings in debug mode (nf-test test */local --profile=~test,docker,debug).
  • 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).

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 8, 2026

This PR is against the main branch ❌

  • Do not close this PR
  • Click Edit and change the base to dev
  • This CI test will remain failed until you push a new commit

Hi @vagkaratzas,

It looks like this pull-request is has been made against the nf-core/proteinannotator main branch.
The main branch on nf-core repositories should always contain code from the latest release.
Because of this, PRs to main are only allowed if they come from the nf-core/proteinannotator dev branch.

You do not need to close this PR, you can change the target branch to dev by clicking the "Edit" button at the top of this page.
Note that even after this, the test will continue to show as failing until you push a new commit.

Thanks again for your contribution!

@vagkaratzas vagkaratzas closed this May 8, 2026
@vagkaratzas vagkaratzas deleted the patch-aws-test-config branch May 8, 2026 09:47
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 8, 2026 •
edited
Loading

nf-core pipelines lint overall result: Failed ❌

Posted for pipeline commit 8b9d647

+| ✅ 227 tests passed       |+
#| ❔   7 tests were ignored |#
!| ❗   1 tests had warnings |!
-| ❌   3 tests failed       |-
Details

❌ Test failures:

❗ Test warnings:

❔ Tests ignored:

  • files_exist - File is ignored: .github/workflows/ci.yml
  • files_exist - File is ignored: conf/igenomes.config
  • files_exist - File is ignored: conf/igenomes_ignored.config
  • files_unchanged - File ignored due to lint config: .github/PULL_REQUEST_TEMPLATE.md
  • files_unchanged - File ignored due to lint config: assets/nf-core-proteinannotator_logo_light.png
  • files_unchanged - File ignored due to lint config: docs/images/nf-core-proteinannotator_logo_light.png
  • files_unchanged - File ignored due to lint config: docs/images/nf-core-proteinannotator_logo_dark.png

✅ Tests passed:

Run details

  • nf-core/tools version 4.0.2
  • Run at 2026-05-08 09:48:28

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@vagkaratzas