-
Notifications
You must be signed in to change notification settings - Fork 23
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
Migrate to PodIRCBot #149
Comments
@Christopher-Chianelli Firstly, thank you for initiating such a huge refactor! Your contributions to this project are greatly appreciated. I personally don't know much about this new architecture, so I will be learning quite a bit as this refactor continues. Looking over your points I don't see many cons, and the pros are substantial. One concern I have though is how this refactor will be done on GitHub. We should keep the project in the |
@cuijulian The https://github.com/Christopher-Chianelli/PodIRCBot is intended to be a base for all containerized IRC Bots; no VileBot1 specific code will be in it. Once it is stable (that is, actually have a proper API, multiple templates and testing), I will either add PodIRCBot at a git submodule or make its scripts work with URL's. Once that done, each function/service VileBot1 provides will be in its own folder (in it own programming language). I recommend the refactor be done on a separate branch (maybe the "containers" branch?) |
@Christopher-Chianelli This makes sense, I misunderstood what you meant by "migrate to", but I think I have a better idea of what you mean. My gut feeling is making I agree with the refactor being done on a separate branch in |
|
Branch created. Feel free to create a PR to it that adds the scripts. |
We have a bot that love containers, yet does not run in a container. I propose we migrate to https://github.com/Christopher-Chianelli/PodIRCBot when it become stable. This have several advantages:
!ka last 0
), the other services remain unaffected.VileBot.java
ugliness; the proxy doesn't need to know about each service, as the service connect itself to the proxy via a static url.The text was updated successfully, but these errors were encountered: