Skip to content
This repository has been archived by the owner on Nov 24, 2023. It is now read-only.

Apply suggestion #63

Merged
merged 3 commits into from
May 2, 2018
Merged

Apply suggestion #63

merged 3 commits into from
May 2, 2018

Conversation

killercup
Copy link
Member

killercup added 2 commits May 2, 2018 00:07
probably because we need to enable clippy lint groups (but I don't want
to spend time on this right now)
@killercup
Copy link
Member Author

Update to fix newer clippy lints not important for compiletest rustfix, will fix later (or remove)

@killercup killercup requested a review from alexcrichton May 2, 2018 19:15
@killercup
Copy link
Member Author

r? @alexcrichton

Want to land this right now so I can whip up the lib/bin split next and publish it all as 0.2?

@killercup
Copy link
Member Author

Wait, after making this build again, of course, sorry, thought I already did this

Looks like they're failing on nightly and hopefully we'll get enough coverage in
rust-lang/rust itself to cover these use cases.
@killercup killercup merged commit 820dda2 into master May 2, 2018
@alexcrichton alexcrichton deleted the apply_suggestion branch May 2, 2018 22:45
alexcrichton pushed a commit to killercup/rust that referenced this pull request May 4, 2018
Uses branch from <rust-lang/rustfix#63>
until we publish a new release.
djrenren pushed a commit to djrenren/compiletest that referenced this pull request Aug 26, 2019
Uses branch from <rust-lang/rustfix#63>
until we publish a new release.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants