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

Coveralls integration is broken #3836

Closed
jsirois opened this issue Sep 5, 2016 · 5 comments
Closed

Coveralls integration is broken #3836

jsirois opened this issue Sep 5, 2016 · 5 comments
Assignees
Labels

Comments

@jsirois
Copy link
Contributor

jsirois commented Sep 5, 2016

Our coveralls integration has been broken for a while. Local experiments show this is due to using coveralls latest to submit reports which depends on the coverage 4 series vs pants itself depending on the coverage 3.7 series and the coverage data file format having changed between these two series.

@jsirois
Copy link
Contributor Author

jsirois commented Mar 6, 2018

Now that #5544 is in there may be hope of resurrecting this effort.

@Eric-Arellano
Copy link
Contributor

Still broken. We removed the plugin from our Readme. We should add back coverage, but we'll want to start from scratch rather than reviving this.

@stuhood
Copy link
Sponsor Member

stuhood commented May 18, 2020

IMO, let's keep this open to track it.

@stuhood stuhood reopened this May 18, 2020
@benjyw benjyw removed infra labels Sep 9, 2021
@thejcannon
Copy link
Member

Should we close this, as Pants v2 doesn't have a coveralls interation?

@Eric-Arellano
Copy link
Contributor

I think so. @stuhood feel free to reopen

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants