-
Notifications
You must be signed in to change notification settings - Fork 13.4k
deprecate vecmap #26734
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
deprecate vecmap #26734
Conversation
r? @huonw (rust_highfive has picked a reviewer for you, use r? to override) |
👍 |
1 similar comment
👍 |
Sounds good to me, but the ownership of the crate on crates.io should probably fall under the contain-rs organization. @GBGamer would you be ok transferring ownership? Otherwise a |
I'm fine with this as well. |
I hacked into crates.io and reverse engineered the 0.1.0 published. |
@bors: r+ 98b95a5c141fc3ae78bda7d687fc087f33cbc54b Not sure how this flew under the radar for so long :( |
⌛ Testing commit 98b95a5 with merge 137d6ca... |
💔 Test failed - auto-mac-64-opt |
@bors r=alexcrichton |
📌 Commit 841b1e6 has been approved by |
@bors r- |
@bors r=alexcrichton |
📌 Commit 2907611 has been approved by |
⌛ Testing commit 2907611 with merge 56fd71b... |
💔 Test failed - auto-mac-64-opt |
@bors: retry On Wed, Jul 29, 2015 at 5:56 PM, bors [email protected] wrote:
|
VecMap doesn't really fit with the current standard library's strategy (small!). I've mirrored the code to https://github.com/contain-rs/vec-map but @GBGamer has already claimed the name on crates.io a couple months ago for the same purpose. It hasn't been updated since, though. CC @rust-lang/libs
VecMap doesn't really fit with the current standard library's strategy (small!).
I've mirrored the code to https://github.com/contain-rs/vec-map
but @GBGamer has already claimed the name on crates.io a couple months ago for the same purpose. It hasn't been updated since, though.
CC @rust-lang/libs