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

Document common changes needed during a 0.x -> 1.x upgrade #205

Merged
merged 1 commit into from Jun 27, 2016

Conversation

@carols10cents
Copy link
Contributor

carols10cents commented Jun 25, 2016

Hi! I had to upgrade another crate from 0.x to 1.x, so I decided to actually write up what I did. Then I went and looked at the crates that @SimonSapin upgraded and found some more. I also spot checked a few reverse dependency crates that have done this upgrade. So I'm sure there are more changes that I'm missing, but these should be the most common ones.

There are still quite a few crates that are using url 0.x listed on crates.io, I hope this will help when they decide to upgrade! 🎉


This change is Reviewable

@metajack
Copy link
Contributor

metajack commented Jun 26, 2016

@carols10cents
Copy link
Contributor Author

carols10cents commented Jun 27, 2016

Closing in favor of #206!

@carols10cents carols10cents deleted the carols10cents:add-upgrading-doc branch Jun 27, 2016
@SimonSapin
Copy link
Member

SimonSapin commented Jun 27, 2016

Since I used your commit as-is let’s let github mark it as merged :)

bors-servo added a commit that referenced this pull request Jun 27, 2016
Add some to UPGRADING.md

This is a few additions on top of #205. Thanks @carols10cents starting this!

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/rust-url/206)
<!-- Reviewable:end -->
@carols10cents carols10cents restored the carols10cents:add-upgrading-doc branch Jun 27, 2016
@carols10cents carols10cents reopened this Jun 27, 2016
@carols10cents
Copy link
Contributor Author

carols10cents commented Jun 27, 2016

Ok! :)

@bors-servo bors-servo merged commit bdb50a2 into servo:master Jun 27, 2016
0 of 2 checks passed
0 of 2 checks passed
continuous-integration/appveyor/pr Waiting for AppVeyor build to complete
Details
continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
@carols10cents carols10cents deleted the carols10cents:add-upgrading-doc branch Jun 27, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.