-
Notifications
You must be signed in to change notification settings - Fork 20
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
Publish as hex package #17
Comments
I've actually started tinkering an pkg version of this app with some improvements over lessons learnt tho I don't think I can focus on this right now as I'm concurrently distributing time between some projects already. But I can come back to this issue next month for sure. |
If you do I think you should reconsider the Instead some kind of worker pool, for example using HTTP connections are limited after all. |
Hacktoberfest project, @lubien? 😀 |
Hey that's a pretty good project. Wanna go for it? |
I'll do the best I can, but I'm no expert. :) |
I think this project would benefit from being published as a hex package.
IMO the current structure is not fully suited for publishing just yet. For example the global nature of the poller and matcher would make defining multiple bots impossible. As a solution it might make sense to consider an approach similar to an
Ecto.Repo
where configuration lives for a certain repo which then gets supervised and takes care of it's processes.What do you think? I'm maintaining two small libs already so I would be happy to assist.
The text was updated successfully, but these errors were encountered: