Skip to content

Add a from_str function. #8988

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

Closed
wants to merge 3 commits into from
Closed

Add a from_str function. #8988

wants to merge 3 commits into from

Conversation

emberian
Copy link
Member

@emberian emberian commented Sep 5, 2013

It just calls out to the associated function on the trait.

@alexcrichton
Copy link
Member

Is there an associated bug for this? I thought we were normally wary about adding new things to the prelude.

@emberian
Copy link
Member Author

emberian commented Sep 5, 2013

@alexcrichton no, but it's a long-standing complaint that FromStr::from_str("foo") is ugly and bad to use.

I don't feel strongly about this PR.

@brson
Copy link
Contributor

brson commented Sep 5, 2013

Agree that important changes like additions to the prelude should have issues backing them, but I'm +1 on this change since parsing strings is common and currently really ugly.

bors added a commit that referenced this pull request Sep 8, 2013
It just calls out to the associated function on the trait.
@bors bors closed this Sep 8, 2013
flip1995 pushed a commit to flip1995/rust that referenced this pull request Jun 16, 2022
… r=flip1995

Test metadata collection in Bors CI workflow

This PR adds a new check to bors CI workflows, which ensures that the metadata collection success, when it's run as part of the `deploy` script. I've only added it to bors workflows, as the runtime will be high while it'll also succeed most of the time. This is a preparation for rust-lang/rust-clippy#8947.

---

changelog: none

r? `@ghost`
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.

4 participants