Skip to content

Conversation

@GSPatton
Copy link
Contributor

@GSPatton GSPatton commented Apr 13, 2023

This PR adds Ruby 3.2 support in preparation for Puppet 8 and removes Ruby 2.5 and Puppet 6 support

@GSPatton GSPatton changed the title (WIP)(CONT-807) committing Gemfile for debugging (CONT-807) Add Ruby 3.2 support Apr 17, 2023
@GSPatton GSPatton marked this pull request as ready for review April 17, 2023 09:52
@GSPatton GSPatton requested a review from a team as a code owner April 17, 2023 09:52
Copy link
Member

@david22swan david22swan left a comment

Choose a reason for hiding this comment

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

One small comment

ref: "2.6.0"
```

Only install the `yumrepo_core` module when testing against Puppet 6 or greater:
Copy link
Member

Choose a reason for hiding this comment

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

Is this really necessary now that everything below Puppet 7 is unsupported?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

i originally removed all of this but thought it still had a place since it still seems relevant to >6. Seems now that this module should always be installed since all supported puppets are >6

Copy link
Member

Choose a reason for hiding this comment

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

I guess it's not a big thing

@david22swan david22swan merged commit b743ebe into main Apr 17, 2023
@david22swan david22swan deleted the CONT-807-add_ruby_3 branch April 17, 2023 10:36
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