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

Convert the Legacy Puppet functions to the Modern format #1260

Open
david22swan opened this issue Jul 26, 2022 · 4 comments
Open

Convert the Legacy Puppet functions to the Modern format #1260

david22swan opened this issue Jul 26, 2022 · 4 comments

Comments

@david22swan
Copy link
Member

david22swan commented Jul 26, 2022

Since the release of the Modern Puppet function format several of the functions and all new ones to be added have been written in it, however the vast majority of the original functions remain in the Legacy format.

While they are still functional and will not currently cause any errors in their use, they are still outdated and using a format that has been fazed out, potential causing potential problems down the line.

As such it would be best to update them now before any issues can arise.

@b4ldr
Copy link
Collaborator

b4ldr commented Aug 1, 2022

What's your view on if they should be name spaced with stdlib::. My personal view is that stdlib is a bit more special and unique then most modules as such we could drop the requirement to namespace all (new or refactored) functions

@ekohl
Copy link
Collaborator

ekohl commented Aug 1, 2022

Somehow this doesn't link back to #1259 but that's where this came up. In #1259 (comment) I stated pretty much the same as @b4ldr that I do feel stdlib is special and can be in the global namespace.

@github-actions
Copy link

Hello! 👋

This issue has been open for a while and has had no recent activity. We've labelled it with attention-needed so that we can get a clear view of which issues need our attention.

If you are waiting on a response from us we will try and address your comments on a future Community Day.

Alternatively, if it is no longer relevant to you please close the issue with a comment.

@alexjfisher
Copy link
Collaborator

This still needs doing...

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

No branches or pull requests

4 participants