Skip to content

Conversation

@thaJeztah
Copy link
Member


Bumps codecov/codecov-action from 4 to 5.


updated-dependencies:

  • dependency-name: codecov/codecov-action dependency-type: direct:production update-type: version-update:semver-major ...

- What I did

- How I did it

- How to verify it

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

Bumps [codecov/codecov-action](https://github.com/codecov/codecov-action) from 4 to 5.
- [Release notes](https://github.com/codecov/codecov-action/releases)
- [Changelog](https://github.com/codecov/codecov-action/blob/main/CHANGELOG.md)
- [Commits](codecov/codecov-action@v4...v5)

---
updated-dependencies:
- dependency-name: codecov/codecov-action
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah
Copy link
Member Author

FWIW: I have not yet looked into the "token-less" upload. It looks like that doesn't automatically work on protected branches, so may need some looking at; #5620 (comment)

@codecov-commenter
Copy link

codecov-commenter commented Nov 19, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 59.49%. Comparing base (81401f3) to head (b0c0cd5).
Report is 14 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #5636   +/-   ##
=======================================
  Coverage   59.49%   59.49%           
=======================================
  Files         346      346           
  Lines       29371    29371           
=======================================
  Hits        17474    17474           
  Misses      10922    10922           
  Partials      975      975           
---- 🚨 Try these New Features:

uses: codecov/codecov-action@v5
with:
file: ./build/coverage/coverage.txt
files: ./build/coverage/coverage.txt
Copy link
Member Author

Choose a reason for hiding this comment

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

Files takes a comma-separated list.. bit of an odd choice for this (not sure why they didn't pick an array / slice)

@thaJeztah thaJeztah requested a review from crazy-max November 19, 2024 12:56
@thaJeztah
Copy link
Member Author

Ah! Hitting that issue that should be fixed through #5631

#16 CANCELED
------
 > [internal] load build context:
------
ERROR: failed to solve: Internal: rpc error: code = Internal desc = received 4294967294-bytes data exceeding the limit 491940 bytes
Build references
Check build summary support
Error: buildx bake failed with: ERROR: failed to solve: Internal: rpc error: code = Internal desc = received 4294967294-bytes data exceeding the limit 491940 bytes

@thaJeztah thaJeztah merged commit 7b67057 into docker:master Nov 21, 2024
@thaJeztah thaJeztah deleted the bump_codecov_action branch November 21, 2024 16:52
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.

4 participants