Skip to content

Conversation

stefankiesz
Copy link
Contributor

@stefankiesz stefankiesz commented Mar 18, 2024

What was changed?
The git tag of the Producer C branch used as dependency library.

Why was it changed?
The tests were run using the CMake cleanup branch of Producer C

How was it changed?
By changing the git tag of the Producer C branch.

What testing was done for the changes?
The CI was run with the new dependency branch.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@sirknightj sirknightj added build test-run CI uses Git Secrets, which are only accessible by maintainers. labels Apr 28, 2024
@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 75.21%. Comparing base (f6bc9fc) to head (176fcb7).
Report is 9 commits behind head on develop.

❗ Current head 176fcb7 differs from pull request most recent head 1e0270d. Consider uploading reports for the commit 1e0270d to get more accurate results

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #1952      +/-   ##
===========================================
- Coverage    75.44%   75.21%   -0.24%     
===========================================
  Files           49       49              
  Lines        13930    13875      -55     
===========================================
- Hits         10510    10436      -74     
- Misses        3420     3439      +19     

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

Copy link

This is a very old issue. We encourage you to check if this is still an issue in the latest release and if you find that this is still a problem, please feel free to open a new one.

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

Labels

build test-run CI uses Git Secrets, which are only accessible by maintainers.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants