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

Fix repository URL for AppVeyor #384

Merged
merged 1 commit into from Oct 17, 2017
Merged

Fix repository URL for AppVeyor #384

merged 1 commit into from Oct 17, 2017

Conversation

@mgeisler
Copy link
Contributor

mgeisler commented Jul 25, 2017 •
edited by larsbergstrom

For some reason, this project is not registered under servo/rust-url on AppVeyor.

This change is Reviewable
@mgeisler
Fix repository URL for AppVeyor
Loading status checks…
f546438 
For some reason, this project is not registered under `servo/rust-url` on AppVeyor.
@jdm
Copy link
Member

jdm commented Jul 25, 2017

The appveyor webhook did not like this change; appveyor returned a 404 response when it was notified about this PR.
@mgeisler
Copy link
Contributor Author

mgeisler commented Jul 25, 2017

The appveyor webhook did not like this change; appveyor returned a 404 response when it was notified about this PR.

Oh, that's strange? I don't think the Cargo configuration has any influence on AppVeyor?
@jdm
Copy link
Member

jdm commented Jul 25, 2017

Oh, I totally misread the PR. That's doubly interesting.
@mgeisler
Copy link
Contributor Author

mgeisler commented Jul 25, 2017

Hehe, okay :-) I cannot see the repository settings and the failed web hook -- all I know is that the README links to https://ci.appveyor.com/project/Manishearth/rust-url and that going to https://ci.appveyor.com/project/servo/rust-url doesn't work.
@mbrubeck
Copy link
Contributor

mbrubeck commented Oct 17, 2017

@bors-servo r+

I'm going to proceed on the assumption that the Appveyor webhook error was a weird coincidence.
@bors-servo
Copy link
Contributor

bors-servo commented Oct 17, 2017

📌 Commit f546438 has been approved by mbrubeck
@bors-servo
Copy link
Contributor

bors-servo commented Oct 17, 2017

⌛ Testing commit f546438 with merge 766622a...
bors-servo added a commit that referenced this pull request Oct 17, 2017
@bors-servo
Auto merge of #384 - mgeisler:patch-1, r=mbrubeck
Loading status checks…
766622a 
Fix repository URL for AppVeyor

For some reason, this project is not registered under `servo/rust-url` on AppVeyor.

<!-- 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/384)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Oct 17, 2017

☀️ Test successful - status-travis
Approved by: mbrubeck
Pushing 766622a to master...
@bors-servo bors-servo merged commit f546438 into servo:master Oct 17, 2017
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
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
@mgeisler @jdm @mbrubeck @bors-servo
You can’t perform that action at this time.