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

Deprecate _ms functions that predate the Duration API #29604

Merged
merged 1 commit into from
Nov 6, 2015

Conversation

bstrie
Copy link
Contributor

@bstrie bstrie commented Nov 5, 2015

No description provided.

@rust-highfive
Copy link
Collaborator

r? @aturon

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

@bstrie
Copy link
Contributor Author

bstrie commented Nov 5, 2015

Given that aturon and acrichto seemed surprised to discover that these APIs had not been deprecated, would we want to backport these to beta? If so, then hold off approving this until I have the chance to change the "since" tags.

@sfackler
Copy link
Member

sfackler commented Nov 5, 2015

Doesn't really seem worth backporting. Since we missed these in 1.4, it doesn't seem to matter a ton if we deprecate in 1.5 or 1.6.

@bstrie
Copy link
Contributor Author

bstrie commented Nov 5, 2015

@sfackler Doesn't matter a ton, but it does add an additional six week period where people may use these functions accidentally.

@Gankra
Copy link
Contributor

Gankra commented Nov 5, 2015

We backport very little. Basically only serious bug fixes. I don't care if people use this functionality, it's not exactly a footgun or anything, just a crufty interface.

@steveklabnik
Copy link
Member

Yup, big 👎 to back porting.

On Nov 5, 2015, at 04:51, Alexis Beingessner notifications@github.com wrote:

We backport very little. Basically only serious bug fixes. I don't care if people use this functionality, it's not exactly a footgun or anything, just a crufty interface.


Reply to this email directly or view it on GitHub.

@alexcrichton
Copy link
Member

I agree with others that it's ok to not backport this, we unfortunately missed the 1.5 train but another's always on the way! Regardless, though, thanks @bstrie!

@bors: r+ a2efa2a

@bors
Copy link
Contributor

bors commented Nov 6, 2015

⌛ Testing commit a2efa2a with merge 6878fce...

bors added a commit that referenced this pull request Nov 6, 2015
@bors bors merged commit a2efa2a into rust-lang:master Nov 6, 2015
@brson brson added the relnotes Marks issues that should be documented in the release notes of the next release. label Nov 14, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
relnotes Marks issues that should be documented in the release notes of the next release.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants