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

Puppet8 #1339

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

Puppet8 #1339

wants to merge 10 commits into from

Conversation

davidphay
Copy link

Pull Request Checklist

Description

Update and support Stdlib 9 & puppet8

Related Issue

Fixes # .

Motivation and Context

Since Puppet 8 has been release, a lot of module use stdlib 9 but currently this module doesn't support it because it use a removed function (exists)

How Has This Been Tested?

Test on a Puppet 8 server on 2 different OS (debian 11 & CentOS 9)

General

  • Update README.md with any necessary configuration snippets

  • New parameters are documented

  • New parameters have tests

  • Tests pass - bundle exec rake validate lint spec

@ManishaKumari295 ManishaKumari295 self-assigned this May 7, 2024
@yorickps
Copy link

Can the maintainers have a look at this PR? Puppet 8 has been released for some time now.

@ManishaKumari295
Copy link

Can the maintainers have a look at this PR? Puppet 8 has been released for some time now.

Hello @yorickps , can you please have a look at this branch - https://github.com/sensu/sensu-puppet/tree/manisha_1339 and let us know if your concern is solved with this.

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

Successfully merging this pull request may close these issues.

3 participants