8000 Replace support for keil with documentation entry by Spacetown · Pull Request #727 · gcovr/gcovr · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Replace support for keil with documentation entry #727

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

Spacetown
Copy link
Member
@Spacetown Spacetown commented Feb 21, 2023

Since the #623 only adds support for some markers and #697 showed that the files generated by Keil uVision compiler are not compatible with the gcov specification this PR undo the changes from #623 and adds a entry to the cookbook how the files can be transformed.

Closes #697

@Spacetown Spacetown added Type: Enhancement Gcov Docs related to the documentation or website labels Feb 21, 2023
@Spacetown Spacetown added this to the 6.0 milestone Feb 21, 2023
@Spacetown Spacetown requested a review from latk February 21, 2023 20:48
@Spacetown Spacetown force-pushed the replace_support_for_keil_with_documentation_entry branch from 8f78139 to 4220df2 Compare February 21, 2023 20:49
@codecov
Copy link
codecov bot commented Feb 21, 2023

Codecov Report

Base: 95.58% // Head: 95.57% // Decreases project coverage by -0.01% ⚠️

Coverage data is based on head (513bb51) compared to base (9c3771c).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #727      +/-   ##
==========================================
- Coverage   95.58%   95.57%   -0.01%     
==========================================
  Files          28       28              
  Lines        3850     3845       -5     
  Branches      672      671       -1     
==========================================
- Hits         3680     3675       -5     
  Misses         92       92              
  Partials       78       78              
Flag Coverage Δ
ubuntu-20.04 94.27% <100.00%> (-0.01%) ⬇️
windows-2019 95.17% <100.00%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
gcovr/gcov_parser.py 97.61% <100.00%> (-0.02%) ⬇️
gcovr/workers.py 98.83% <0.00%> (-0.04%) ⬇️
gcovr/tests/test_gcovr.py 97.64% <0.00%> (ø)
gcovr/writer/html.py
gcovr/writer/html/__init__.py 95.45% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@Spacetown Spacetown requested a review from latk February 22, 2023 18:42
@Spacetown Spacetown merged commit ea2f842 into gcovr:master Feb 22, 2023
@Spacetown Spacetown deleted the replace_support_for_keil_with_documentation_entry branch February 22, 2023 21:40
D-Walther pushed a commit to D-Walther/gcovr that referenced this pull request Jun 16, 2025
* Undo changes from gcovr#621 since this only adds partial support.
* Add Cookbook entry.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Docs related to the documentation or website Gcov Type: Enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Backwards compatibility for older gcov branch syntax to support uVision Keil coverage output.
2 participants
0