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

Array repeat expression lengths must be monomorphic at MIR building time #68285

Merged
merged 1 commit into from Jan 17, 2020

Conversation

@oli-obk
Copy link
Contributor

oli-obk commented Jan 16, 2020

fixes #67743

@rust-highfive

This comment has been minimized.

Copy link
Collaborator

rust-highfive commented Jan 16, 2020

r? @davidtwco

(rust_highfive has picked a reviewer for you, use r? to override)

Copy link
Member

davidtwco left a comment

LGTM, r=me when CI passes

@oli-obk

This comment has been minimized.

Copy link
Contributor Author

oli-obk commented Jan 16, 2020

@bors r=davidtwco

@bors

This comment has been minimized.

Copy link
Contributor

bors commented Jan 16, 2020

📌 Commit eed0d33 has been approved by davidtwco

@oli-obk

This comment has been minimized.

Copy link
Contributor Author

oli-obk commented Jan 17, 2020

@bors p=1 this unblocks the playground allowing it to update its nightly

@bors

This comment has been minimized.

Copy link
Contributor

bors commented Jan 17, 2020

⌛️ Testing commit eed0d33 with merge 71c6346...

bors added a commit that referenced this pull request Jan 17, 2020
Array repeat expression lengths must be monomorphic at MIR building time

fixes #67743
@bors

This comment has been minimized.

Copy link
Contributor

bors commented Jan 17, 2020

☀️ Test successful - checks-azure
Approved by: davidtwco
Pushing 71c6346 to master...

@bors bors added the merged-by-bors label Jan 17, 2020
@bors bors merged commit eed0d33 into rust-lang:master Jan 17, 2020
5 checks passed
5 checks passed
homu Test successful
Details
pr Build #20200116.34 succeeded
Details
pr (Linux mingw-check) Linux mingw-check succeeded
Details
pr (Linux x86_64-gnu-llvm-7) Linux x86_64-gnu-llvm-7 succeeded
Details
pr (Linux x86_64-gnu-tools) Linux x86_64-gnu-tools succeeded
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

4 participants
You can’t perform that action at this time.