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

Conversation

@killercup
Copy link
Contributor

No description provided.

@killercup killercup requested review from Manishearth and ehuss July 16, 2019 19:49
@killercup
Copy link
Contributor Author

Hm, 2 tests fail. Probably because of changes in the diagnostic output. I'll into those separately and rebase this later.

@ehuss
Copy link
Collaborator

ehuss commented Jul 17, 2019

Let me know if you need help with them.

Also, can this line be removed? It doesn't look like it really needs nightly, and that's the only thing requiring it. Probably doesn't matter much if you are guaranteeing the tests only pass on nightly, though, I was just surprised (and the error was not clear). Might also be good to have a note somewhere (docs or an assert in code) about the required rustc version, so people like me don't get confused by errors using the wrong version.

@ehuss
Copy link
Collaborator

ehuss commented Aug 6, 2019

I posted #172 to update the tests.

@alexcrichton alexcrichton merged commit 1736be4 into rust-lang:master Aug 7, 2019
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.

3 participants