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

Jump to bottom

Add support for constant generics #204

Merged
merged 1 commit into from Mar 18, 2020
Merged

Add support for constant generics #204

merged 1 commit into from Mar 18, 2020

Conversation

c410-f3r
Copy link
Contributor

No breaking changes and no internal hacks. This PR only adds a feature to switch between implementations of the Array trait.

I personally think that this approach is pretty reasonable and should be at least considered for discussion.

@mbrubeck
Copy link
Collaborator

Cool, thanks! It looks like the .travis.yml file needs some updates to make the tests pass in CI, but otherwise this looks great.

@mbrubeck
Copy link
Collaborator

@bors-servo r+

@bors-servo
Copy link
Contributor

📌 Commit 70e967e has been approved by mbrubeck

@bors-servo
Copy link
Contributor

⌛ Testing commit 70e967e with merge 9cdf8fb...

@bors-servo
Copy link
Contributor

☀️ Test successful - checks-travis
Approved by: mbrubeck
Pushing 9cdf8fb to master...

@bors-servo bors-servo merged commit 9cdf8fb into servo:master Mar 18, 2020
@c410-f3r
Copy link
Contributor Author

Thank you! Thank you very much!

@mbrubeck
Copy link
Collaborator

Let me know if you need this published to crates.io soon; otherwise we will probably wait until we have a few more changes before making a new release.

@c410-f3r
Copy link
Contributor Author

I am fine using the git version and now that this change is upstream, everything is cool. Thank you one more time.

mbrubeck added a commit to mbrubeck/rust-smallvec that referenced this pull request Apr 6, 2020
@mbrubeck
Version 1.3.0
851d222 
* Add a new unstable `const_generics` feature (servo#204).
* Improve inlining of constructor functions (servo#206).
* Add a `slice.to_smallvec()` convenience method (servo#203).
* Documentation and testing improvements.
@mbrubeck mbrubeck mentioned this pull request Apr 6, 2020
Merged
bors-servo added a commit that referenced this pull request Apr 8, 2020
@bors-servo
Auto merge of #207 - mbrubeck:bump, r=mbrubeck
3957cd8 
Version 1.3.0

* Add a new unstable `const_generics` feature (#204).
* Improve inlining of constructor functions (#206).
* Add a `slice.to_smallvec()` convenience method (#203).
* Documentation and testing improvements.
@c410-f3r c410-f3r mentioned this pull request Apr 19, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@c410-f3r @mbrubeck @bors-servo