-
Notifications
You must be signed in to change notification settings - Fork 27
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 CI #56
Comments
@cheelee as discussed please add push access to this repo for machine user @openworm-bot |
Done. Does it work? |
yes but I think as per https://help.github.com/en/articles/repository-permission-levels-for-an-organization#repository-access-for-each-permission-level I think it'll need admin access to this repo to modify webhooks. Hopefully the list of permissions doesn't look to scary to you (esp if you can just enable it only for this repo) |
Ok I've set it to the highest level of permissions for the repo - Maintain. |
hmm interesting - can't it be changed to admin? The circleci.com list of repos I can enable CI on using the bot account is currently empty |
Maybe I do not have those permissions. @slarson - are you able to assign admin permissions to bot accounts specific to just this repo? @casperdcl - are you aware of what specific capabilities the account must have to perform its tasks? When I set it to Maintain, it indicated the account would have the permission to push and change repo settings. |
Ok nevermind, for some reason I didn't see there was another entry below Maintain earlier. I have set it to Admin for the repo now. |
kewl yes unfortunately as per the table on https://help.github.com/en/articles/repository-permission-levels-for-an-organization#repository-access-for-each-permission-level it has to be admin for webhooks :/ |
Ok wait. I am a little confused now. Does that mean @openworm-bot will need to be added to the organization as a member? |
@casperdcl I am trying to solve the problem you had earlier with providing admin privileges for webhooks. I am unsure of what the solution is other than adding @openworm-bot as a collaborator for a specific repository with Admin privileges. It seems as each repo has its own webhooks. |
@balicea yes that's right. I'd think org member with as little privileges as possible, plus admin on the docs repo. Could just do admin collaborator if you prefer. Also let me know how you'd like me to send you guys the login details for the bot. |
managed to set up the webhook and even made a quick configuration (#57) |
add continuous integration testing.
Presumably currently the site is built automatically by readthedocs.org (without build status feedback on GitHub). Adding our own CI tests means:
The text was updated successfully, but these errors were encountered: