Skip to content

Comments

feat(76583): integrate code coverage#44

Open
bogdandina wants to merge 14 commits intoaks-devfrom
feature/76583_integrate-code-coverage
Open

feat(76583): integrate code coverage#44
bogdandina wants to merge 14 commits intoaks-devfrom
feature/76583_integrate-code-coverage

Conversation

@bogdandina
Copy link
Contributor

No description provided.

@codecov
Copy link

codecov bot commented Feb 22, 2026

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

@chihaiaalex
Copy link

would be nice to split java version upgrade from code coverage integration.

jobs:
build:
uses: HSLdevcom/transitdata-shared-workflows/.github/workflows/ci-cd-java.yml@main
uses: HSLdevcom/transitdata-shared-workflows/.github/workflows/ci-cd-java.yml@feature/59649-measure-test-coverage

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

will you change this feature/59649-measure-test-coverage?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, after the transitdata-shared-workflows PR will be merged and a new release will be available.

@@ -0,0 +1,11 @@
<settings xmlns="http://maven.apache.org/SETTINGS/1.0.0"

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

how does it work locally?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These settings are copy&pasted in the Docker image and they are needed for being able to build & execute tests inside a Docker container.
Please check out how the actual Dockerfile is looking like.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants