feat: add back Codecov with all files

[thomasrockhu-codecov]

@caendesilva sorry for taking so long, I have an idea on how to get 94ef842 working. I don't actually know how to run the GitHub Actions in this PR, but if possible, I'd like to see if this clears up the issue from https://community.codecov.com/t/codecov-not-discovering-files-when-uploaded-in-ci/3691/2

[caendesilva]

Hi there @thomasrockhu-codecov! I actually managed to get Codecov to work in the new monorepo https://github.com/hydephp/develop/. I actually have no idea why it works there though. I'll see if I can enable your run so we can try to debug this.

[caendesilva]

I'm not seeing anything in the recent commits section. https://app.codecov.io/gh/hydephp/framework

[thomasrockhu-codecov]

@caendesilva probably would help if we were actually collecting coverage 😬. Pushed a new commit, hopefully this one works.

[caendesilva]

@caendesilva probably would help if we were actually collecting coverage 😬. Pushed a new commit, hopefully this one works.

Whoops! I'm having a ton of issues with GitHub at the moment, which I think it's why some runs failed, but I can't access the logs. I see that we need to re-add a code coverage driver. I'll see if GitHub lets me add it.

Edit: yeah some tests failed due to GitHub being down so the repo couldn't be cloned. One is complaining that we don't have a code coverage driver. I left a review where I think it needs to go.

[thomasrockhu-codecov]

@caendesilva yeah, looks like we are having troubles with GitHub too. I pushed the change and some log statements

[caendesilva]

@caendesilva yeah, looks like we are having troubles with GitHub too. I pushed the change and some log statements

Alright cool! I approved the workflow. Hoping it works this time!

[thomasrockhu-codecov]

@caendesilva yikes, moved the logs after the Codecov step

[caendesilva]

Yeah I'm a bit stumped as to what's happening. If it helps, we don't need to put the reports in a subdirectory as we are not pushing any commits to the repository.

[thomasrockhu-codecov]

@caendesilva ok what if we remove the directory-specific search and try all the arguments

[caendesilva]

@caendesilva ok what if we remove the directory-specific search and try all the arguments

Let's try it!

[codecov]

Codecov Report

Merging #558 (2e97759) into master (1694d9f) will decrease coverage by 9.09%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##            master     #558      +/-   ##
===========================================
- Coverage   100.00%   90.90%   -9.10%     
===========================================
  Files            1        3       +2     
  Lines           10       33      +23     
===========================================
+ Hits            10       30      +20     
- Misses           0        3       +3     

Impacted Files	Coverage Δ
a/framework/framework/hyde/app/bootstrap.php	90.90% <0.00%> (ø)
...er/work/framework/framework/hyde/app/bootstrap.php	90.90% <0.00%> (ø)
app/bootstrap.php	90.90% <0.00%> (ø)

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

[caendesilva]

Codecov Report

Merging #558 (e49ca31) into master (1694d9f) will decrease coverage by 9.09%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##            master     #558      +/-   ##
===========================================
- Coverage   100.00%   90.90%   -9.10%     
===========================================
  Files            1        1              
  Lines           10       11       +1     
===========================================
  Hits            10       10              
- Misses           0        1       +1     

Impacted Files Coverage Δ
hyde/app/bootstrap.php 90.90% <0.00%> (ø)
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 1e15613...e49ca31. Read the comment docs.

Oh yeah, awesome!

[thomasrockhu-codecov]

@caendesilva woo, ok we're uploading. I'm trying some path fixes here that might help get all of the coverage

[caendesilva]

@caendesilva woo, ok we're uploading. I'm trying some path fixes here that might help get all of the coverage

Woo awesome!

[thomasrockhu-codecov]

@caendesilva, ok, for the coverage reports being uploaded, I'm really only seeing this app/bootstrap.php file. This looks true in CI/CD too (see this run). Are we missing a step to collect coverage for all files?

[caendesilva]

@caendesilva, ok, for the coverage reports being uploaded, I'm really only seeing this app/bootstrap.php file. This looks true in CI/CD too (see this run). Are we missing a step to collect coverage for all files?

It's possible the runner needs to be reconfigured. It's pretty late here, but I'll take a look at it tomorrow. Thank you so so much for your help so far @thomasrockhu-codecov!

[caendesilva]

Since everything seems to be working, I'll go ahead and merge this PR and fix the path in a separate PR. Once again, thank you so much!

By the way, I lost access to the Codecov account I originally used when I converted @hydephp to an organization. So I won't be able to mark it as resolved, but from my end it all looks great, so you can mark it as resolved if you want. Thanks a bunch!

[thomasrockhu-codecov]

@caendesilva sounds good! Let me know if you run into any issues that I can help with.