-
Notifications
You must be signed in to change notification settings - Fork 18
(CONT-808) Ruby 3 / Puppet 8 Support #48
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
Conversation
0efacf1
to
ff97982
Compare
7af50b2
to
67d9af1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm wondering if it's better to first drop Puppet 2 and 3 support from the codebase before doing RuboCop changes.
bf3b68d
to
4544e7c
Compare
@ekohl All valid suggestions here, thank you 😄 . With regards to removal of legacy Puppet code, I think ultimately it's going to come down to time, for me at least. I'd be happy to work through PRs from yourself / other community members though. This is something I've just asked about internally as a way to get rspec-puppet rolling again. |
47f0bed
to
79f7f56
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good to me. Will wait for another review
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
This PR adds support for Ruby3x and Puppet 8