-
Notifications
You must be signed in to change notification settings - Fork 92
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
Consider making each package its own go module #52
Comments
@jcchavezs - I agree, I was thinking the same. How will someone do a |
We have had that idea. That JWT lib problem triggered that idea for me personally. At the moment we do not have clear plan how and when it should be done (probably with v5). Anyway I support the idea. one note here. as a maintainer it quite inconvenient to track many repos and their issues/PRs. The amount of questions/false-positives is quite high. Sometimes github does not send even notifications so I find existance of some PR only if I specially check that repository PR tab. |
It works as long as no circular dependencies or ideally no dependencies between them otherwise it will be a nightmare.
I think (99% sure) introducing go modules won't be a breaking change as people will still call packages in the same way, the only difference will be how the dependencies are resolved so I think no need to wait for v5. |
replying to myself: semantic versioning of each MW would not be possible probably in this setup. |
Not really but that is IMHO a goal which is not worth. Semantic versioning of each MW opens up the box for a huge compatibility matrix which you might not like to deal with. I'd just release them all together. |
Go 1.17 has some nice features
I have not tried it yet but seems that from 1.17 dependencies you are not using are not downloaded anymore. |
That would be awesome and would allow keeping it all together here with simpler versioning. |
Currently this project includes way too many dependencies as it is a single module. Making all of them a different module would avoid users to pull dependencies that aren't important for their use case e.g. pulling jaeger and zipkin when willing to use prometheus. Does that make sense?
The text was updated successfully, but these errors were encountered: