8000 Add function overview to html details by Spacetown · Pull Request #554 · gcovr/gcovr · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Add function overview to html details #554

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 16 commits into from
Mar 25, 2022
Merged

Add function overview to html details #554

merged 16 commits into from
Mar 25, 2022

Conversation

Spacetown
Copy link
Member

If a detailed report is generated a overview page with alphabetical ordered table of all functions is created. The lines are linked to the details page where the function is defined.

@Spacetown Spacetown added this to the 5.1 milestone Jan 9, 2022
@Spacetown Spacetown requested a review from latk January 9, 2022 20:29
@codecov
Copy link
codecov bot commented Jan 9, 2022 •
edited
Loading

Codecov Report

Merging #554 (6bffdd3) into master (b2fb2c5) will decrease coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #554      +/-   ##
==========================================
- Coverage   95.76%   95.72%   -0.05%     
==========================================
  Files          23       23              
  Lines        3331     3344      +13     
  Branches      610      613       +3     
==========================================
+ Hits         3190     3201      +11     
- Misses         69       70       +1     
- Partials       72       73       +1
Flag Coverage Δ
ubuntu-18.04 94.58% <ø> (-0.04%) ⬇️
windows-2019 95.40% <ø> (-0.05%) ⬇️

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

Impacted Files Coverage Δ
gcovr/writer/html.py 95.49% <0.00%> (-0.42%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b2fb2c5...6bffdd3. Read the comment docs.

@Spacetown Spacetown marked this pull request as draft February 20, 2022 21:36
@Spacetown Spacetown marked this pull request as ready for review February 26, 2022 22:56
@Spacetown Spacetown force-pushed the add_function_overview_to_html_details branch from f335a13 to ddc140e Compare February 27, 2022 20:24
@Spacetown Spacetown mentioned this pull request Mar 14, 2022
Closed
@Spacetown Spacetown force-pushed the add_function_overview_to_html_details branch from 2640467 to 8acf530 Compare March 24, 2022 20:32
latk
latk approved these changes Mar 25, 2022
View reviewed changes
Copy link
Member
@latk latk left a comment

Choose a reason for hiding this comment

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

The function list works and is good 👍

I have two comments about the HTML5 commit, though.

@Spacetown Spacetown force-pushed the add_function_overview_to_html_details branch from 56ffa51 to 6bffdd3 Compare March 25, 2022 20:38
@Spacetown Spacetown merged commit 5c4d60b into gcovr:master Mar 25, 2022
@Spacetown Spacetown deleted the add_function_overview_to_html_details branch March 25, 2022 20:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@latk latk latk approved these changes

Assignees
No one assigned
Labels
Format: HTML Type: Enhancement
Projects
None yet
Milestone
5.1
Development

Successfully merging this pull request may close these issues.
2 participants
@Spacetown @latk
0