Revert "Bump codecov/codecov-action from 3 to 4"

[jishnub]

Reverts #127

Coverage was not being uploaded correctly in that PR. A Codecov token needs to be added for coverage reports to be uploaded.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (41f58b5) 94.83% compared to head (aa15175) 94.81%.
Report is 5 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #128      +/-   ##
==========================================
- Coverage   94.83%   94.81%   -0.02%     
==========================================
  Files           5        4       -1     
  Lines         445      444       -1     
==========================================
- Hits          422      421       -1     
  Misses         23       23              

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

[giordano]

I'm confused by how this works: according to https://github.com/JuliaMath/AbstractFFTs.jl/actions/runs/7784505365/job/21225171516?pr=128#step:8:31 upload failed

[jishnub]

This fails sporadically if there's no Codecov token added to the repo, perhaps due to rate limits. Re-running the job should fix the upload issue.

Some of the other jobs in the run have successfully uploaded their reports.

[devmotion]

#129 fixes the codecov integration by using a token, I think this might be the better approach than reverting the codecov bump.