Skip to content

skip coverage upload to codecov on contributors repositories#3966

Merged
filiphr merged 1 commit into
mapstruct:mainfrom
hduelme:skip-coverage-upload-to-codecov-on-contributors-repositories
Jan 1, 2026
Merged

skip coverage upload to codecov on contributors repositories#3966
filiphr merged 1 commit into
mapstruct:mainfrom
hduelme:skip-coverage-upload-to-codecov-on-contributors-repositories

Conversation

@hduelme

@hduelme hduelme commented Dec 31, 2025

Copy link
Copy Markdown
Contributor

Like in mapstruct/mapstruct-idea#252, upload coverage to codecov is failing from contributors repositories, see https://github.com/hduelme/mapstruct/actions/runs/20571229857/job/59078754665, with 429 - Rate limit reached.

I disabled the upload for contributors repositories, see https://github.com/hduelme/mapstruct/actions/runs/20622913081/job/59228250813

To ensure correct upload the action should be updated and a token should be added https://docs.codecov.com/docs/adding-the-codecov-token#github-actions. In addition the fail_ci_if_error flag could be set.

@filiphr filiphr merged commit a055213 into mapstruct:main Jan 1, 2026
4 checks passed
@filiphr filiphr added the build label Jan 1, 2026
@filiphr filiphr added this to the 1.7.0 milestone Jan 1, 2026
@filiphr

filiphr commented Jan 1, 2026

Copy link
Copy Markdown
Member

Thanks @hduelme.

To ensure correct upload the action should be updated and a token should be added https://docs.codecov.com/docs/adding-the-codecov-token#github-actions. In addition the fail_ci_if_error flag could be set.

Are you referring to something like in the IntelliJ plugin repo? The token is on the organization level, so we only need to update the workflow. I'm not sure when I'll have access to a laptop, but I can merge a PR easily

@hduelme

hduelme commented Jan 1, 2026

Copy link
Copy Markdown
Contributor Author

@filiphr I was referring to the action config in the IntelliJ plugin repo. In #3967 I updated the action config accordingly and coverage is published again.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants