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

Add grunt task to make webhook creation simple #5

Open
patcon opened this issue Mar 5, 2014 · 6 comments
Open

Add grunt task to make webhook creation simple #5

patcon opened this issue Mar 5, 2014 · 6 comments

Comments

@patcon
Copy link
Member

patcon commented Mar 5, 2014

Right now, there's manual configuration. Would be great to make it simpler.

@parkr
Copy link
Member

parkr commented Mar 5, 2014

Let's do this via hubot!

/notify me <repo> <events>

@patcon
Copy link
Member Author

patcon commented Mar 5, 2014

Ah, we could still add that, but I wouldn't personally want to give hubot the admin permissions via API to do that from chat...! I'm trying to avoid giving hubot admin access to repos, in all honesty :)

Can we leave this one as grunt and add another ticket for a hubot command?

@parkr
Copy link
Member

parkr commented Mar 5, 2014

Are you worried about hubot having access, your hubot host (e.g. heroku) having admin access, or your colleagues having access? If the latter, we can utilize auth.coffee, if the other two, then we're limiting what hubot can automate for us. I'd be 👍 on a system which allowed for both – either do it locally or give Hubot access to ze 🌐!

@parkr
Copy link
Member

parkr commented Mar 5, 2014

Related: I use the git-at-me wizard, which automates 100% of this for you: https://github.com/parkr/all-your-github-are-belong-to-us/blob/master/script/generator

@patcon
Copy link
Member Author

patcon commented Mar 5, 2014

@parkr it's more that I don't think I can ask to give hubot admin access to the gittip/www.gittip.com repo, since we're dealing with money and I'm not ready to ask for that much responsibility yet :)

But that project looks cool! Let's just add both. Can this issue be for the grunt task?

@parkr
Copy link
Member

parkr commented Mar 5, 2014

Ah, fair. Yeah, I'll spike out a different issue for the command.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants