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

rust bindgen support #279

Merged
merged 1 commit into from Mar 28, 2016
Merged

rust bindgen support #279

merged 1 commit into from Mar 28, 2016

Conversation

@larsbergstrom
Copy link
Contributor

larsbergstrom commented Mar 26, 2016

r? @Manishearth @edunham

I'm still not sure of the best way to handle the fact that we need to hand out a GH webhook secret to arbitrary people to set this up. I suspect we will need to set up homu.servo.org at some point.


This change is Reviewable

@aneeshusa
Copy link
Member

aneeshusa commented Mar 26, 2016

Do we hand out the same GH webhook secret for every repo, or do we generate a new one for each repo?

@nox nox closed this Mar 26, 2016
@nox nox reopened this Mar 26, 2016
@nox
Copy link
Member

nox commented Mar 26, 2016

Misclick. -_-

@larsbergstrom
Copy link
Contributor Author

larsbergstrom commented Mar 26, 2016

@aneeshusa We only have one hook right now.

We could create and store new tokens for each repo (homu definitely supports per-repo tokens!), but I think I'd really like to get it installed and integrated as a first-class app via GitHub so that interaction through salt is not required to add a new repo.

@edunham
Copy link
Contributor

edunham commented Mar 28, 2016

@bors-servo
Copy link
Contributor

bors-servo commented Mar 28, 2016

📌 Commit 1188860 has been approved by edunham

@bors-servo
Copy link
Contributor

bors-servo commented Mar 28, 2016

Testing commit 1188860 with merge 98cfe8e...

bors-servo added a commit that referenced this pull request Mar 28, 2016
rust bindgen support

r? @Manishearth @edunham

I'm still not sure of the best way to handle the fact that we need to hand out a GH webhook secret to arbitrary people to set this up. I suspect we will need to set up homu.servo.org at some point.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/saltfs/279)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Mar 28, 2016

☀️ Test successful - travis

@bors-servo bors-servo merged commit 1188860 into servo:master Mar 28, 2016
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
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

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