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

Add support for rustfix coverage tracking #167

Merged
merged 2 commits into from Jul 16, 2019

Conversation

@phansch
Copy link
Contributor

@phansch phansch commented Mar 26, 2019

Pretty much a copy of rust-lang/rust#59398 and
rust-lang/rust#59677 minus the rustc bootstrap
things.

This will allow rustfix coverage tracking in Clippy and other
libraries that emit MachineApplicable diagnostics.

phansch added 2 commits Mar 26, 2019
Pretty much a copy of rust-lang/rust#59398 minus
the rustc bootstrap things.

This will allow rustfix coverage tracking in Clippy and other
libraries that emit `MachineApplicable` diagnostics.
@phansch phansch closed this Apr 16, 2019
@phansch phansch reopened this Apr 16, 2019
@phansch
Copy link
Contributor Author

@phansch phansch commented Apr 24, 2019

Let me know if there's something I can help out with. It would be nice to get this merged so we can update rust-lang/rust-clippy#3630.

@phansch
Copy link
Contributor Author

@phansch phansch commented Jul 16, 2019

@laumann would you be up for merging this and releasing a new version? It would be a big help for getting good rustfix test coverage in Clippy

edit: woops, Manish was faster, sorry about the ping =)

@Manishearth Manishearth merged commit ffdcb7f into Manishearth:master Jul 16, 2019
2 checks passed
@phansch phansch deleted the rustfix_coverage branch Jul 16, 2019
@laumann
Copy link
Collaborator

@laumann laumann commented Jul 18, 2019

@phansch Sorry for being so tardy, compiletest has gotten away from me a little bit.

@laumann
Copy link
Collaborator

@laumann laumann commented Jul 18, 2019

Was a new version released? 0.3.22 looks to be from April, so should this go in 0.3.23?

@phansch
Copy link
Contributor Author

@phansch phansch commented Jul 18, 2019

@laumann No worries at all.

I think for this particular PR no new version is required because it's enough to update the rustfix coverage issue using the compiletest-rs master branch temporarily.

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

Successfully merging this pull request may close these issues.

None yet

3 participants