Codecov publish fix [HZ-5302]

[ihsandemir]

Codecov publish fix. Avoiding to do this caused the publish action find and use extra coverage files for the test files which were excluded when running the gcov.

If you look at the current run and download the coverage_report.zip you can see that the test directory is actually excluded but it is published differently to codecov.

The details on the config option disable_search is here more details here.

fixes #1308

Here is the proof that this works. The test folder is not included.

[JackPGreen]

and even when I add the workflow_dispatch: it did not work, I can not see manual trigger button.

try adding pull_request instead and it’ll trigger on your PR. I’d like to see this fix “working” because I don’t have context on the cause of the problem - where did this fix come from? Did someone else suggest it?

[codecov-commenter]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.61%. Comparing base (529eac0) to head (d71899b).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #1381      +/-   ##
==========================================
+ Coverage   81.10%   88.61%   +7.51%     
==========================================
  Files         216      161      -55     
  Lines       34777    19559   -15218     
  Branches     2044        0    -2044     
==========================================
- Hits        28207    17333   -10874     
+ Misses       6364     2226    -4138     
+ Partials      206        0     -206     

☔ 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.

[ihsandemir]

and even when I add the workflow_dispatch: it did not work, I can not see manual trigger button.

try adding pull_request instead and it’ll trigger on your PR. I’d like to see this fix “working” because I don’t have context on the cause of the problem - where did this fix come from? Did someone else suggest it?

Just changed it and there is a run result already.