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

(CAT-2254) Remove puppet 7 / Ruby 2.7 related code #368

Merged
merged 4 commits into from
Mar 12, 2025

Conversation

LukasAud
Copy link
Contributor

@LukasAud LukasAud commented Mar 10, 2025

Following the EOL of Puppet 7, we are updating the codebase to remove Puppet 7 related code. Changes made include the removal of Ruby 2.7 testing in our Actions matrix and Rubocop bumps to newer versions, alongside a new TargetRuby setting and several autocorrections, and some outdated version conditional removals.

This PR will enable #367 to be merged

Following the EOL of Puppet 7, we are updating the codebase to
remove Puppet 7 related code. Automated testing, rubocop pins and
ruby 2.7 related code is removed amongst other things.
@LukasAud LukasAud added the maintenance Maintenance (PDK sync and other maintaining commits) label Mar 10, 2025
@LukasAud LukasAud requested review from a team as code owners March 10, 2025 16:17
@LukasAud LukasAud marked this pull request as draft March 10, 2025 16:18
@LukasAud LukasAud force-pushed the CAT-2254-Remove_puppet_7_support branch 2 times, most recently from fb6fe7d to ff485c9 Compare March 11, 2025 16:22
@LukasAud LukasAud force-pushed the CAT-2254-Remove_puppet_7_support branch from ff485c9 to 7a87173 Compare March 12, 2025 07:50
@LukasAud LukasAud marked this pull request as ready for review March 12, 2025 07:51
@joshcooper joshcooper merged commit d4b6849 into main Mar 12, 2025
4 checks passed
@joshcooper joshcooper deleted the CAT-2254-Remove_puppet_7_support branch March 12, 2025 18:23
@LukasAud LukasAud added backwards-incompatible and removed maintenance Maintenance (PDK sync and other maintaining commits) labels Mar 13, 2025
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