Skip to content

Fix missing len function#127

Merged
kenyon merged 2 commits into
voxpupuli:mainfrom
anthonysomerset:fix-missing-function
Oct 21, 2025
Merged

Fix missing len function#127
kenyon merged 2 commits into
voxpupuli:mainfrom
anthonysomerset:fix-missing-function

Conversation

@anthonysomerset

@anthonysomerset anthonysomerset commented Oct 21, 2025

Copy link
Copy Markdown
Contributor

Pull Request (PR) description

640078c brought in additional logic but seems to reference a non-existent function len

assuming this should actually be the core function length so updating to reference this

i am unsure if this is the desired output but my nodes are all working just fine with this change now

also updated the metadata to reference the correct https://github.com/voxpupuli/puppet-patching_as_code repo - as the forge page currently takes users to a non-existent repo url at present

This Pull Request (PR) fixes the following issues

none reported yet

@kenyon kenyon added the bug Something isn't working label Oct 21, 2025

@kenyon kenyon left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Someone had Python on the brain 😉 https://docs.python.org/3/library/functions.html#len

@kenyon kenyon merged commit 9cd9af6 into voxpupuli:main Oct 21, 2025
4 checks passed
@anthonysomerset anthonysomerset deleted the fix-missing-function branch October 22, 2025 07:13
@anthonysomerset

Copy link
Copy Markdown
Contributor Author

GIven that the module is broken in the 3.0.0 release please can we bump out 3.0.1 asap :)

@kenyon

kenyon commented Oct 22, 2025

Copy link
Copy Markdown
Member

GIven that the module is broken in the 3.0.0 release please can we bump out 3.0.1 asap :)

Let me see if I can get the modulesync in #125 merged so that we can make a release.

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

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants