Skip to content

Testing code coverage workflow - #18469

Closed
pcanal wants to merge 42 commits into
root-project:v6-36-00-patchesfrom
pcanal:code_coverage
Closed

Testing code coverage workflow #18469
pcanal wants to merge 42 commits into
root-project:v6-36-00-patchesfrom
pcanal:code_coverage

Conversation

@pcanal

@pcanal pcanal commented Apr 22, 2025

Copy link
Copy Markdown
Member

This Pull request:

Changes or fixes:

Checklist:

  • tested changes locally
  • updated the docs (if necessary)

This PR fixes #

@pcanal pcanal self-assigned this Apr 22, 2025
@pcanal pcanal closed this Apr 22, 2025
@pcanal pcanal reopened this Apr 22, 2025
@ferdymercury

Copy link
Copy Markdown
Collaborator

Maybe related: #18392

@pcanal
pcanal changed the base branch from master to v6-36-00-patches April 23, 2025 14:29
@pcanal pcanal closed this Apr 23, 2025
@pcanal pcanal reopened this Apr 23, 2025
@codecov

codecov Bot commented Apr 23, 2025

Copy link
Copy Markdown

Codecov Report

Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Please upload report for BASE (v6-36-00-patches@83143ba). Learn more about missing BASE report.

Files with missing lines Patch % Lines
core/meta/src/TClass.cxx 0.00% 1 Missing ⚠️
Additional details and impacted files
@@                 Coverage Diff                 @@
##             v6-36-00-patches   #18469   +/-   ##
===================================================
  Coverage                    ?    8.07%           
===================================================
  Files                       ?     3696           
  Lines                       ?   560947           
  Branches                    ?   258188           
===================================================
  Hits                        ?    45277           
  Misses                      ?   483892           
  Partials                    ?    31778           
Flag Coverage Δ
unittests 8.07% <0.00%> (?)

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

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@pcanal
pcanal force-pushed the code_coverage branch 4 times, most recently from 34c3923 to 7972f79 Compare April 24, 2025 18:03
@pcanal pcanal closed this Apr 25, 2025
@pcanal pcanal reopened this Apr 25, 2025
@pcanal pcanal closed this Apr 25, 2025
@pcanal pcanal reopened this Apr 25, 2025
@pcanal
pcanal force-pushed the code_coverage branch 3 times, most recently from b09087d to 31200db Compare April 29, 2025 17:20
@pcanal pcanal closed this Apr 29, 2025
@pcanal pcanal reopened this Apr 29, 2025
@pcanal
pcanal force-pushed the code_coverage branch 4 times, most recently from 9462b41 to abc0c2f Compare April 30, 2025 20:05
pcanal added 24 commits April 30, 2025 15:31
Somehow the coverage tarball fails because of the softlink ...
Solve the warning:
.../core/meta/src/TClass.cxx: In member function 'virtual void TDumpMembers::Inspect(TClass*, const char*, const char*, const void*, Bool_t)':
.../core/meta/src/TClass.cxx:770:20: warning: 'char* strncpy(char*, const char*, size_t)' specified bound 994 equals destination size [-Wstringop-truncation]
  770 |             strncpy(line + kvalue, *ppointer, kline - kvalue);
      |             ~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
@pcanal

pcanal commented May 13, 2025

Copy link
Copy Markdown
Member Author

Migrated to master @ #18709

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants