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

Setting up AppVeyor for an org #226

Closed
jakirkham opened this issue Jan 3, 2018 · 5 comments
Closed

Setting up AppVeyor for an org #226

jakirkham opened this issue Jan 3, 2018 · 5 comments
Labels
maintenance Work needed by a maintainer
Milestone

Comments

@jakirkham
Copy link
Member

Unlike Travis CI or Coveralls, the AppVeyor setup for an org is a little different. Namely we need to have a dedicate org email, which could use the GMail + trick or similar. Then users can be added via this administrator account. More details in this doc.

@jakirkham
Copy link
Member Author

As this would be the same org account for numcodes as zarr, no need to open this issue again on that repo.

@alimanfoo
Copy link
Member

Hi @jakirkham, I think AppVeyor should be set up, but I am seeing an issue with the webhook which means that builds are not getting triggered for PRs. It looks like the webhook POST for a "push" event is getting through fine, but the POST for "pull request" events is returning 403, with this in the message body:

{"message":"HTTP GET to https://api.github.com/repos/zarr-developers/zarr/pulls/227 returned status code 403: Forbidden\nFull response body:\n{\"message\":\"API rate limit exceeded for alimanfoo.\",\"documentation_url\":\"https://developer.github.com/v3/#rate-limiting\"}"}

Is this anything you've seen before? I googled around a bit but can't find an obvious solution.

@alimanfoo alimanfoo added this to the v2.2 milestone Jan 4, 2018
@alimanfoo alimanfoo added the maintenance Work needed by a maintainer label Jan 4, 2018
@alimanfoo
Copy link
Member

alimanfoo commented Jan 4, 2018

Scratch that, looks like AppVeyor has righted itself (#227).

@jakirkham
Copy link
Member Author

I have seen that issue before, but mostly in the context of conda-forge feedstocks where we actually do go over the GitHub rate limit sometimes and have taken measures to address that.

Can't see why you would have run into the issue here though. The GitHub rate limit resets every hour. So even if that issue does come up, it will eventually go away. The fact that it came up at all is what is interesting.

We should keep an eye on it and see if it reoccurs in the future.

@jakirkham
Copy link
Member Author

AppVeyor setup LGTM. So closing this out.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance Work needed by a maintainer
Projects
None yet
Development

No branches or pull requests

2 participants