Skip to content

/rust

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

Decide if Wrapping/WrappingOps should be in the prelude #23454

Closed
alexcrichton opened this Issue Mar 17, 2015 · 5 comments

Comments

Assignees

@aturon aturon

Labels
None yet
Projects
None yet
Milestone
  1.0 beta
3 participants
@alexcrichton @aturon @pnkfelix
@alexcrichton
Copy link
Member

alexcrichton commented Mar 17, 2015

The associated RFC does not mention these two items being in the prelude, and we should probably decide whether or not they should be there.
@alexcrichton

This comment has been minimized.

Sign in to view
Copy link
Member Author

alexcrichton commented Mar 17, 2015

triage: I-needs-decision, P-backcompat-libs (1.0 beta)

@alexcrichton alexcrichton added A-libs I-needs-decision labels Mar 17, 2015

@alexcrichton alexcrichton added this to the 1.0 beta milestone Mar 17, 2015

@alexcrichton alexcrichton referenced this issue Mar 20, 2015

Merged

Stabilize std::num #23549

@aturon

This comment has been minimized.

Sign in to view
Copy link
Member

aturon commented Mar 20, 2015

cc @nikomatsakis @brson @huonw
@aturon

This comment has been minimized.

Sign in to view
Copy link
Member

aturon commented Mar 20, 2015

cc @pnkfelix

@alexcrichton alexcrichton assigned aturon Mar 24, 2015

@pnkfelix

This comment has been minimized.

Sign in to view
Copy link
Member

pnkfelix commented Mar 26, 2015

Decision: We're not going to put these things into the prelude.

So they need to be removed from the prelude.

@pnkfelix pnkfelix removed the I-needs-decision label Mar 26, 2015

@alexcrichton

This comment has been minimized.

Sign in to view
Copy link
Member Author

alexcrichton commented Apr 1, 2015

Resolved in #23549

@alexcrichton alexcrichton closed this Apr 1, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.