Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 76.85%. Comparing base (a1772ab) to head (d58d4a7).
Report is 1 commits behind head on dev.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Good idea! But I am not sure this is going to work as is, I think we are only able to comment on a PR when the action trigger is pull_request_target, that's why we have a different YAML file for the other actions such as branch protection or linting

good point. In the docs they mention adding:

permissions:
  pull-requests: write

have we tested if that works?

I did go through the rabbit hole some time ago and the only solution was to create a separate yaml file, but worth giving it a try with the testpipeline 👍

looks like it works: nf-core/testpipeline#109 (comment)

or does it need to come from a fork. one second

ah, you were right 😞

@nf-core-bot changelog

This change was introduced with 511126a so the PR can be closed

Sorry, I didn't notice this was in the nf-test.yml file! Reopening

@mashehu what is the status of this PR? I think the downgrade of nf-schema is not necessary, but printing the summary of tests is still relevant