Skip to content
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

fog requirement is overbroad #315

Open
jspaleta opened this issue Dec 28, 2018 · 1 comment
Open

fog requirement is overbroad #315

jspaleta opened this issue Dec 28, 2018 · 1 comment

Comments

@jspaleta
Copy link

Only fog subpackage used is fog-aws

We should be able to narrow down the runtime dep to 'fog-aws' instead of 'fog'

The toplevel 'fog' gem brings in a ton of other stuff that isn't needed.

Working on this in my fork now, Will close as part of PR.

@majormoses
Copy link
Member

Ya that would be good and should be low hanging fruit. I think its only for an LB check, which we have several of...perhaps we should consolidate them instead.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants