pdksync - (MAINT) - Require Stdlib 9.x#921
Conversation
|
PR currently blocked by main CI failing. Issues are not related to change afaics |
aa054ec to
fd40b54
Compare
fd40b54 to
7bfda16
Compare
|
I think the PR title should be adjusted. this now doesn't allow stdlib 9, it requires it (which is in my opinion a breaking change). |
smortex
left a comment
There was a problem hiding this comment.
Looks like some namespaced function is missing. No idea what is going wrong with some versions of windows at this point.
|
These windows failure seems quite reproducible. I have basically no experience with docker and windows, but it looks like the command we run do not match the ones from upstream documentation: vs. Can't say if we do this by mistake or on purpose. |
|
This PR is failing due to problems present on main. This does not create any new failures on top on the already existing ones afaik. We should be able to merge this and address failures on main in a different place. |
f462aa1 to
23b1bf9
Compare
This deprecated function was removed from Puppet. The Puppet 4 type system can be used to ensure the validity of parameters with better error reporting.
Co-authored-by: Romain Tartière <romain@blogreen.org>
23b1bf9 to
bfc3422
Compare
(MAINT) - Update Stdlib dependency
pdk version:
2.7.0