Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Uploaded coverage reports don't take into account original repo #291

Open
j-mie6 opened this issue Jan 27, 2021 · 0 comments
Open

Uploaded coverage reports don't take into account original repo #291

j-mie6 opened this issue Jan 27, 2021 · 0 comments
Assignees
Labels
bug Something isn't working

Comments

@j-mie6
Copy link

j-mie6 commented Jan 27, 2021

Describe the bug
The reports submitted to code climate for PRs made from forks and the master branch on the original repo itself have indistinguishable names.

Version of codeclimate-action you're using
v2.7.5

Expected behavior
When I have a PR made against my repo from a fork codeclimate's diff-coverage runs. To do that it requires a coverage report from both master on my repo, and the PR itself. However, when the PR comes from a forked repo, and that repo has made its PR from it's own master, the reports submitted override eachother:

image

Should instead be something like this:

image

The consequence is that codeclimate won't run the diff checking, because it is just seeing multiple test coverage reports submitted to master. I'm not sure exactly what's expected here: whether it should be master and minut1bc/master or something else.

Additional context
I've raised an issue here just in case it is actually codeclimate's fault, but I suspect that the problem may be here instead, where the names are generated for submission.

@j-mie6 j-mie6 added the bug Something isn't working label Jan 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants