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

Add doc comment for percent_encoding::from_hex #155

Merged
merged 1 commit into from Dec 26, 2015
Merged

Add doc comment for percent_encoding::from_hex #155

merged 1 commit into from Dec 26, 2015
+18 −0

Conversation

@cmbrandenburg
Copy link
Contributor

cmbrandenburg commented Dec 26, 2015

Review on Reviewable
@cmbrandenburg
Add doc comment for percent_encoding::from_hex
Loading status checks…
be7530e
@Ms2ger
Copy link
Contributor

Ms2ger commented Dec 26, 2015

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

bors-servo commented Dec 26, 2015

📌 Commit be7530e has been approved by Ms2ger
@bors-servo
Copy link
Contributor

bors-servo commented Dec 26, 2015

⌛ Testing commit be7530e with merge 2ae8dfe...
bors-servo added a commit that referenced this pull request Dec 26, 2015
@bors-servo
Auto merge of #155 - cmbrandenburg:from_hex_doc, r=Ms2ger
Loading status checks…
2ae8dfe 
Add doc comment for percent_encoding::from_hex

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/rust-url/155)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Dec 26, 2015

☀️ Test successful - travis
@bors-servo bors-servo merged commit be7530e into servo:master Dec 26, 2015
3 checks passed
3 checks passed
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
@SimonSapin
Copy link
Member

SimonSapin commented Dec 26, 2015

I don't think this was intended to be a public API, but oh well
@cmbrandenburg cmbrandenburg deleted the cmbrandenburg:from_hex_doc branch Dec 26, 2015
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

4 participants
@cmbrandenburg @Ms2ger @bors-servo @SimonSapin
You can’t perform that action at this time.