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

fix: switch to namespaced function stdlib::ensure_packages; require puppetlabs/stdlib 9 #83

Merged
merged 1 commit into from
Oct 17, 2023

Conversation

bastelfreak
Copy link
Member

This is a rebase of #76 . I also updated the metadata.json to address the new required version.

@pcfens pcfens merged commit 778864b into voxpupuli:master Oct 17, 2023
@bastelfreak bastelfreak deleted the ensure branch October 17, 2023 13:48
@bastelfreak
Copy link
Member Author

@pcfens thanks for merging! Can you provide a new release please?

@pcfens
Copy link
Contributor

pcfens commented Oct 17, 2023

Just released as 2.5.0.

Do you think that Vox Pupuli would be interested in picking up this module? We haven't moved away from Puppet, but new stuff typically is run in Kubernetes, so our usage has dropped significantly. I'm happy to discuss here or offline if you think that'd be a worthwhile move.

@bastelfreak
Copy link
Member Author

hey, I think a migration is a good idea. You can keep access to the module but benefit from our CI setup that we manage. I invited you to another team in the vox pupuli github org and you should be able to migrate the repo. Let me know if you've any team members that should have access to the module as well.

@pcfens
Copy link
Contributor

pcfens commented Oct 22, 2023

Thanks for the help. It's been moved.

One reason I wanted to move is because I'm the only person at my organization that works on this and it needs more attention as I know it's widely used. Let me know if there's anything I need to do to finish up migration. For questions about another module is email the best option?

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

Successfully merging this pull request may close these issues.

3 participants