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

Move percent encoding to its own crate #362

Merged
merged 3 commits into from Jun 13, 2017
Merged

Move percent encoding to its own crate #362

merged 3 commits into from Jun 13, 2017

Conversation

@SimonSapin
Copy link
Member

SimonSapin commented Jun 13, 2017 •
edited by larsbergstrom

This is a rebase of #347 with some additional changes. Original work by @seanmonstar.

Fixes #347.

This change is Reviewable
seanmonstar and others added 3 commits May 30, 2017
@seanmonstar @SimonSapin
move percent_encoding to its own crate
9d36a39
@SimonSapin
Don’t use a single-file src directory for percent_encoding
4d0054b
@SimonSapin
Dash in cargo package name
Loading status checks…
dfaacf0
@SimonSapin
Copy link
Member Author

SimonSapin commented Jun 13, 2017

@bors-servo r+
@bors-servo
Copy link
Contributor

bors-servo commented Jun 13, 2017

📌 Commit dfaacf0 has been approved by SimonSapin
@bors-servo
Copy link
Contributor

bors-servo commented Jun 13, 2017

⌛ Testing commit dfaacf0 with merge b054045...
bors-servo added a commit that referenced this pull request Jun 13, 2017
@bors-servo
Auto merge of #362 - servo:percent-crate, r=SimonSapin
Loading status checks…
b054045 
Move percent encoding to its own crate

This is a rebase of #347 with some additional changes. Original work by @seanmonstar.

Fixes #347.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/rust-url/362)
<!-- Reviewable:end -->
@SimonSapin SimonSapin mentioned this pull request Jun 13, 2017
Closed
@bors-servo
Copy link
Contributor

bors-servo commented Jun 13, 2017

☀️ Test successful - status-travis
Approved by: SimonSapin
Pushing b054045 to master...
@bors-servo bors-servo merged commit dfaacf0 into master Jun 13, 2017
5 of 6 checks passed
5 of 6 checks passed
continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
continuous-integration/appveyor/branch AppVeyor build succeeded
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
dependency-ci Dependencies checked
Details
homu Test successful
Details
@SimonSapin SimonSapin deleted the percent-crate branch Jun 13, 2017
@SimonSapin SimonSapin restored the percent-crate branch Jun 13, 2017
@SimonSapin SimonSapin deleted the percent-crate branch Aug 23, 2018
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

3 participants
@SimonSapin @bors-servo @seanmonstar
You can’t perform that action at this time.