Skip to content

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 March 26, 2019 20:31
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 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 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
@phansch phansch deleted the rustfix_coverage branch July 16, 2019 20:21
@laumann
Copy link
Collaborator

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 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 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

Development

Successfully merging this pull request may close these issues.

3 participants