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

puppetlabs_spec_helper: Require 7.X #560

Merged
merged 1 commit into from Apr 2, 2024
Merged

Conversation

bastelfreak
Copy link
Contributor

We're using this since some time at Vox Pupuli and I think Puppet should also use the latest version of their own software. Tested for development group in https://github.com/puppetlabs/puppetlabs-apt/blob/main/Gemfile#L26

@bastelfreak bastelfreak requested a review from a team as a code owner February 15, 2024 11:37
@bastelfreak
Copy link
Contributor Author

@david22swan
Copy link
Member

Sorry for the wait on the merge for this, while testing it out it was discovered it causes failures on windows-2019 and fedora-36 when building the PDK

@bastelfreak
Copy link
Contributor Author

Is that a public pipeline?

@david22swan
Copy link
Member

david22swan commented Feb 20, 2024

Ah, disregard the above comment sorry, ran a clean image and the failures have migrated there. Just unfortunate timing on their appearance it seems.
They showed up just as I began testing your code.

Regardless will continue to work at them and come back and try and get this merged once I can get the pipeline green.
Sorry again for the mistake

@bastelfreak
Copy link
Contributor Author

@david22swan is there any update on this? I need the newer puppetlabs_spec_helper for a few customers.

@david22swan
Copy link
Member

david22swan commented Mar 12, 2024

@bastelfreak Still working on failures on the PDK sorry.
The original ones I last mentioned have been resolved, however new ones appeared following that.

@bastelfreak
Copy link
Contributor Author

What kind of errors? something we can help with?

@david22swan
Copy link
Member

@bastelfreak Sorry for the wait on this, but we've got the PDK back into working order so I am happy to merge.
I've already done some preliminary testing so if you can resolve the conflict that has appeared during the wait I can merge immeaditely.

@bastelfreak
Copy link
Contributor Author

@david22swan rebased it

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.

LGTM

@david22swan david22swan merged commit 7641b71 into puppetlabs:main Apr 2, 2024
3 checks passed
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.

None yet

3 participants