Add support for send-notifications in codecov-action

[ndmanvar]

Currently you need to use cli/binary to send-notifications in GitHub Action.
Codecov GitHub Action isn't setup to call/pass send-notifications to underlying cli.

Ask here would be to have functionality (and supporting docs) for an action to manually notify codecov of upload completion.

[ryanhiebert]

I would appreciate this. I'm trying to use OIDC, and I want to manually tell codecov when all my coverage-uploading jobs have completed, so doing all this through the action would be optimal. I'm not sure if I can use OIDC through a manually-installed CLI, and I'd rather not dig in to find out, but I'd absolutely use it if I could do it here in the action.

[ascopes]

This would be appreciated by me too, my pipelines run a number of parallel builds and aggregate coverage using codecov. Each build targets a slightly different platform where coverage varies due to platform specific code being under test.

I'd like to defer notifications for codecov until these builds have all finished to prevent my email inbox being spammed by comments that immediately get updated again as each build finishes.

[thomasrockhu-codecov]

This has been implemented, you can specify run_command: send-notifications to do so

[ascopes]

Are there docs on how this should look?

[thomasrockhu-codecov]

@ascopes as of yet no, but it would presumably look something like

- name: Send notifications from Codecov
  uses: codecov/codecov-action@v5
  with:
    fail_ci_if_error: true
    run_command: send-notifications
    sha: {{ the sha in question }}
    token: ${{ secrets.CODECOV_TOKEN }}
    verbose: true

[ascopes]

Thanks, will give it a go now.

[ascopes]

Looks like this attaches coverage to the CI pipeline itself, but it doesn't notify pull requests in a comment.

Is this intentional? This would be my main use case that I'd like to support with this.

Example 1: ascopes/protobuf-maven-plugin#535

Example 2: ascopes/java-compiler-testing#781

[ascopes]

@thomasrockhu-codecov please can we re-open this?

[thomasrockhu-codecov]

@ascopes the PR did not get a comment because you marked changes_required: true. This PR does not have any changes to coverage

[ascopes]

Will give it a try with it disabled

[thomasrockhu-codecov]

@ascopes I saw the PR and made a quick note, I think you might actually just want after_n_builds

[ascopes]

Thanks, it is all working now, and yep, just seen that and responded. I'd considered this in the past but it then tightly couples the matrix size to the codecov config which means it is brittle since removing or adding a new JDK or Maven release into the matrix will result in the coverage configuration needing to be updated

[thomasrockhu-codecov]

@ascopes got it, that makes sense!