-
Notifications
You must be signed in to change notification settings - Fork 246
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
Dependencies on gitlab and github should not be defined in the gemspec #382
Comments
I agree however the code is coupled so it would be quite a refactoring. |
I'd say this should result in just moving things around a bit and changing that one place where you require client 3rd-party gems. Technically speaking, it would be a breaking change since people would have to add client gems to their Gemfiles (assuming they don't have them already listed for other reasons than using pronto). |
Yes I was talking about this too in #373 ! |
@emilebosch woah not sure how I missed that! I'm gonna close this one as it's clearly a duplicate. |
First of all, thanks for pronto! It helps us significantly and we enjoy using it.
Now the problem: given that a pronto user may or may not be using GitHub and/or GitLab, the client gems should not be specified in the gemspec because they are not core dependencies.
My specific case is that I use GitHub, so I don't want to have GitLab client gem in my dependencies because it can, and it actually did, cause dependency issues.
One way to solve this problem is to add require statements that rescue from
LoadError
and provide a meaningful error.The text was updated successfully, but these errors were encountered: