Skip to content
This repository was archived by the owner on Nov 19, 2024. It is now read-only.

Conversation

@uhafner
Copy link
Member

@uhafner uhafner commented Oct 8, 2022

Rewrite of the recorder (and step) so that we can use the new Coverage Model. The old step is too tightly coupled with the old model and cannot be refactored so that old pipelines will remain compatible.

Downstream of https://github.com/uhafner/coverage-model/pull/35

@uhafner uhafner added the feature New features label Oct 8, 2022
@uhafner uhafner changed the title Recorder Create a new step to record coverage results Oct 8, 2022
uhafner added 25 commits October 9, 2022 15:06
Use the new parsers in freestyle tests:
- JaCoCo one and two files
- Cobertura single file without complexity
- PIT single file
Add dumb agent and docker agent based tests.
@fo-code
Copy link
Collaborator

fo-code commented Nov 6, 2022

Hi, I just wanted to ask about the status of this pull request. Is there anything I can do to help you with this? :)
I am just wondering because I thought about implementing thresholds for Delta Coverage and an API for accessing these information from other Plugins, but this depends on these model changes here.

@uhafner
Copy link
Member Author

uhafner commented Nov 7, 2022

Superseded by #512.

@uhafner uhafner closed this Nov 7, 2022
@uhafner
Copy link
Member Author

uhafner commented Nov 9, 2022

Hi, I just wanted to ask about the status of this pull request. Is there anything I can do to help you with this? :)
I am just wondering because I thought about implementing thresholds for Delta Coverage and an API for accessing these information from other Plugins, but this depends on these model changes here.

I answered in an email, I hope I used the correct address...

@uhafner uhafner deleted the recorder branch March 27, 2023 14:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

feature New features

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants