-
-
Notifications
You must be signed in to change notification settings - Fork 881
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
Bump puppetlabs/apt dependency #1086
Comments
We are also experiencing a lot of conflicts due to puppet-nginx referencing hard to this old apt module |
How about #1106 as a fix for this? |
make apt a soft dependency per styleguide (resolves #1086)
Sorry, what's the conclusion to this? I face the same problem.
Upgrading the module to 0.7.0 would still be requiring the old module but would not throw the warning? |
@cristianbalan take a look at the changes in: |
Looks perfect @wyardley! Any plan to push that to Forge at some point in the next days? |
make apt a soft dependency per styleguide (resolves voxpupuli#1086)
According to the Puppet Style Guide https://docs.puppet.com/puppet/latest/style_guide.html#dependencies , epel and apt are soft dependencies and should be treated as such.
The text was updated successfully, but these errors were encountered: