Skip to content

Conversation

LukasAud
Copy link
Contributor

@LukasAud LukasAud commented Aug 27, 2025

A set of changes dedicated to implementing puppetcore into our modules. This update, amongst other changes, removes Puppet 7 support.

Checklist

  • 🟢 Spec tests.
  • 🟢 Acceptance tests.

A set of changes dedicated to implementing puppetcore into our modules.
This update, amongst other changes, removes Puppet 7 support.
Copy link
Contributor

@gavindidrichsen gavindidrichsen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@gavindidrichsen gavindidrichsen merged commit 94d8ce8 into main Aug 28, 2025
58 of 109 checks passed
@gavindidrichsen gavindidrichsen deleted the CAT-2353 branch August 28, 2025 07:54
gem "facterdb", '~> 3.0', require: false if Gem::Requirement.create(['>= 3.0.0']).satisfied_by?(Gem::Version.new(RUBY_VERSION.dup))
gem "metadata-json-lint", '~> 4.0', require: false
gem "rspec-puppet-facts", '~> 4.0', require: false
gem "json-schema", '< 5.1.1', require: false
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why did you pin to 5.1.0 or older? Did I introduce bugs in 5.1.1?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That was part of the pdk update command run. It was not a manual adjustment. We can look into updating the templates if the 5.1.1 has significantly important updates.

@LukasAud
Copy link
Contributor Author

Note: All CI failures were GCP related. As discussed internally, we are happy to merge (despite failures) for now while GCP issues are being resolved.

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

Successfully merging this pull request may close these issues.

3 participants