Skip to content

/ rust-url

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

Reduce the number of unstable APIs used #88

Merged
merged 2 commits into from Mar 5, 2015
Merged

Reduce the number of unstable APIs used #88

merged 2 commits into from Mar 5, 2015

Conversation

@alexcrichton
Copy link
Contributor

alexcrichton commented Mar 5, 2015

I was able to get rid of the collections feature entirely and it looks like one of the core features, Str, isn't necessary.
alexcrichton added 2 commits Mar 5, 2015
@alexcrichton
Remove usage of the `collections` feature
8d99afb
@alexcrichton
Remove unused `Str` bound
Loading status checks…
50cb799 
This bound requires the `core` feature but doesn't appear to be required in the
`impl` block itself for `fmt::Display`
SimonSapin added a commit that referenced this pull request Mar 5, 2015
@SimonSapin
Merge pull request #88 from alexcrichton/less-features
Loading status checks…
dfede0b 
Reduce the number of unstable APIs used
@SimonSapin SimonSapin merged commit dfede0b into servo:master Mar 5, 2015
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@SimonSapin
Copy link
Member

SimonSapin commented Mar 5, 2015

Thanks!
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
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
@alexcrichton @SimonSapin
You can’t perform that action at this time.