PretendoNetwork / Bandwidth

Official Pretendo Network Discord community bot
GNU Affero General Public License v3.0
3 stars 12 forks source link

[Feature]: GitHub bot #25

Open jonbarrow opened 3 months ago

jonbarrow commented 3 months ago

Checked Existing

What feature do you want to see added?

Connect Bandwidth to GitHub, doing essentially the same thing as what he does on Discord.

Why do you want to have this feature?

Having Bandwidth auto-respond to issues with error code information can be useful at times. It may also be useful to check that a PR template has been correctly filled out, and if not then responding to the PR alerting the contributor of missing steps.

Any other details to share? (OPTIONAL)

Implementing multiple bots here is likely going to warrant a large scale restructure of the codebase. It may even be worth exploring having separate repositories?

ItzSwirlz commented 3 months ago

A GitHub bot would probably run a separate server (I'm saying would because I have no prior experience with GitHub bots), but it might as well also be in this repo. Or if the main goal is to focus on error codes response, as long as it can access https://github.com/pretendonetwork/error-codes (even locally), it should be fine.

I'm willing to get my feet wet into this. It sounds like fun.

jonbarrow commented 3 months ago

I would hold off to be honest, I'm actually going to mark this as unapproved again. While we would like a GitHub bot I'm not sure it makes a ton of sense to put here, now that I think about it. As far as I know, GitHub sends a webhook for all the data we'd need, and I plan to move our webhook consumers to their own server (we already have a dedicated server for this listening for webhooks to send to the #github channel on our Discord server). Having a GitHub bot would probably make more sense there than here

ItzSwirlz commented 3 months ago

um.. I didn't read the message. I'll just leave https://github.com/ItzSwirlz/bitratepretendotest-github-bot (POCs at https://github.com/ItzSwirlz/pretendo-gh-bot-test/pulls) to sit for now.