eclipse-kuksa / kuksa-databroker

A modern in-vehicle VSS (Vehicle Signal Specification) server written in RUST
https://eclipse-kuksa.github.io/kuksa-website/
Apache License 2.0
12 stars 11 forks source link

Codecov handling #11

Closed erikbosch closed 1 month ago

erikbosch commented 3 months ago

We use codecov, see https://github.com/eclipse-kuksa/kuksa-databroker/blob/main/.github/workflows/kuksa_databroker_build.yml

Some problems observed:

      - name: Upload coverage to Codecov
        # To use v4 a token must be specifed by "token: ${{ secrets.CODECOV_TOKEN }}""
        # Uploaded result available at https://app.codecov.io/gh/eclipse-kuksa/kuksa-databroker
        uses: codecov/codecov-action@v3
        with:
          files: lcov.info
SebastianSchildt commented 2 months ago

I am not sure how Eclipse stance on Codecov is, but we are likely not the first project asking for this.

Someone taking this should make a ticket asking for a orga token here https://gitlab.eclipse.org/eclipsefdn/helpdesk/-/issues

(after creating a ticket ping me or @bs-jokri so we could comment with a "virtual" project-lead thumbs up)

erikbosch commented 2 months ago

There have been projects before

https://gitlab.eclipse.org/eclipsefdn/helpdesk/-/issues/4546

I actually do not know what Otterdog supports here, if you just can request/initiate it by adding it to https://github.com/eclipse-kuksa/.eclipsefdn/blob/main/otterdog/eclipse-kuksa.jsonnet

(Or if someone must add/generate they key manually anyway)

erikbosch commented 2 months ago

Upgrade to new version fixed by #31

Upload worked for latest main, see https://app.codecov.io/github/eclipse-kuksa/kuksa-databroker/commits

But still we could discuss bit more how we want to track code coverage. With the current setup we cannot expect codecov to succeed for all pull requests (as it depends on that the source fork has a token or has luck). But even if it succeeds uploading - what do we want to happen if coverage is shrinking. Shall that be acceptable?

Leaving this issue open for now

erikbosch commented 1 month ago

Think this one can be closed for now