Skip to content

Use the -Zui-testing flag for compile-test tests #3587

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 2 commits into from
Dec 28, 2018

Conversation

flip1995
Copy link
Member

Like @petrochenkov suggested in #3581 (comment)

@phansch
Copy link
Member

phansch commented Dec 27, 2018

This looks really great!

@flip1995 flip1995 added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Dec 27, 2018
@bors
Copy link
Contributor

bors commented Dec 27, 2018

☔ The latest upstream changes (presumably #3586) made this pull request unmergeable. Please resolve the merge conflicts.

@phansch
Copy link
Member

phansch commented Dec 28, 2018

LGTM! Let's get this merged before there's even more PRs with potential conflicts.

@bors r+

@bors
Copy link
Contributor

bors commented Dec 28, 2018

📌 Commit d2dbd0b has been approved by phansch

@bors
Copy link
Contributor

bors commented Dec 28, 2018

⌛ Testing commit d2dbd0b with merge f7bdf50...

bors added a commit that referenced this pull request Dec 28, 2018
Use the -Zui-testing flag for compile-test tests

Like `@petrochenkov` suggested in #3581 (comment)
@phansch phansch removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Dec 28, 2018
@bors
Copy link
Contributor

bors commented Dec 28, 2018

☀️ Test successful - status-appveyor, status-travis
Approved by: phansch
Pushing f7bdf50 to master...

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

Successfully merging this pull request may close these issues.

3 participants