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

Fix non namespaced extlib function #131

Merged

Conversation

neomilium
Copy link
Contributor

@neomilium neomilium commented Apr 3, 2020

This commit fixes the default_content, from extlib module, function usage: this non-namespaced function is deprecated since extlib v3 and have been removed in v5.

Please note that releasing a new version could be welcome by users ;-)

BTW, thanks for your work.

This commit fixes the `default_content`, from extlib module, function usage: this non-namespeced function is deprecated since extlib v3 and have been removed in v5.
@bastelfreak bastelfreak added the bug Something isn't working label Apr 4, 2020
@bastelfreak
Copy link
Member

Hi @neomilium, thanks for the fix! Can you please update the minimal version of extlib in https://github.com/voxpupuli/puppet-fail2ban/blob/master/metadata.json#L13 to the one that has the namespaced version?

@bastelfreak bastelfreak added the needs-work not ready to merge just yet label Apr 4, 2020
@alexjfisher alexjfisher removed the needs-work not ready to merge just yet label Apr 4, 2020
@alexjfisher
Copy link
Member

Minimum version is already correct.

@alexjfisher alexjfisher merged commit 42e11e6 into voxpupuli:master Apr 4, 2020
@alexjfisher
Copy link
Member

@neomilium Thanks!

@alexjfisher
Copy link
Member

I've taken a look at what might be blocking a new release. Could you help with #116 (comment) ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants