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

Rebase of various PRs. #169

Merged
merged 4 commits into from
Oct 19, 2019
Merged

Rebase of various PRs. #169

merged 4 commits into from
Oct 19, 2019

Conversation

emilio
Copy link
Member

@emilio emilio commented Oct 19, 2019

This rebases #164, #165 and #166, and addresses my review comments in the first (removing the function given there's an stable alternative in our new minimum required rust version).

Closes #164
Closes #165
Closes #166

KamilaBorowska and others added 4 commits October 19, 2019 04:41
This is designed to merged after #162, as it deprecates
a function that should be no longer necessary on newer
Rust versions.
This is a backwards incompatible change. VecLike was
deprecated in 0.6.0 released 2 years ago, and that should
have given plenty of time to remove its usages.
@emilio
Copy link
Member Author

emilio commented Oct 19, 2019

@bors-servo r+

@bors-servo
Copy link
Contributor

📌 Commit b3fbc21 has been approved by emilio

@bors-servo
Copy link
Contributor

⌛ Testing commit b3fbc21 with merge 924683a...

bors-servo pushed a commit that referenced this pull request Oct 19, 2019
Rebase of various PRs.

This rebases #164, #165 and #166, and addresses my review comments in the first (removing the function given there's an stable alternative in our new minimum required rust version).

Closes #164
Closes #165
Closes #166
@bors-servo
Copy link
Contributor

☀️ Test successful - checks-travis
Approved by: emilio
Pushing 924683a to master...

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.

None yet

3 participants