Skip to content

Conversation

mark-wiemer
Copy link
Member

@mark-wiemer mark-wiemer commented Aug 24, 2025

PR Checklist

Overview

Codecov provides the same functionality in much the same way, just a different backing company and a different coverage dashboard UX. This PR updates relevant artifacts:

  • use Codecov task
  • use Codecov coverage badge in readme
  • link to Codecov dashboard from mochajs.org
  • adds instructions for how to set up Codecov with OIDC for contributors' forks

Validations (in progress):

@mark-wiemer
Copy link
Member Author

Closing this in favor of migrating to Codecov, see original issue #5436

@mark-wiemer mark-wiemer closed this Sep 7, 2025
@mark-wiemer mark-wiemer changed the title chore: mark coverage upload CI job as optional chore: switch from Coveralls to Codecov Oct 11, 2025
@mark-wiemer mark-wiemer reopened this Oct 11, 2025
@mark-wiemer mark-wiemer marked this pull request as draft October 11, 2025 01:53
@mark-wiemer
Copy link
Member Author

Notes:

  • Triggered from PR isn't logging the coverage token and failing accordingly. This mostly aligns with the docs: "They are not passed to workflows that are triggered by a pull request from a fork." But this isn't from a fork
  • Coveralls did not use a secret, we probably had some link in an account on their side
  • I'm sure this is a common issue that I'll figure out soon enough :)

Copy link

codecov bot commented Oct 12, 2025

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

@MarkWiemer
Copy link

MarkWiemer commented Oct 12, 2025

@mark-wiemer mark-wiemer marked this pull request as ready for review October 12, 2025 05:26
@mark-wiemer
Copy link
Member Author

(@MarkWiemer is my alt acct not a fraud I promise)

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

🛠 Repo: Coveralls outages causing CI to fail

2 participants