puppetlabs / puppetlabs-stdlib

Puppet Labs Standard Library module
http://forge.puppetlabs.com/puppetlabs/stdlib
Apache License 2.0
348 stars 580 forks source link

re-add support for loading aliases in yaml files #1362

Closed lollipopman closed 1 year ago

lollipopman commented 1 year ago

Ruby 3.1 includes Psych 4.0 which no longer loads aliases by default. Since aliases are part of the yaml spec, it seems sensible to re-add support.

puppet-community-rangefinder[bot] commented 1 year ago

loadyaml is a function

Breaking changes to this file MAY impact these 2 modules (near match): * [maany-simple_grid](https://github.com/WLCG-Lightweight-Sites/simple_grid_puppet_module) * [kobybr-apmserver](https://github.com/kobybr/puppet-apmserver.git)

This module is declared in 318 of 580 indexed public Puppetfiles.


These results were generated with Rangefinder, a tool that helps predict the downstream impact of breaking changes to elements used in Puppet modules. You can run this on the command line to get a full report.

Exact matches are those that we can positively identify via namespace and the declaring modules' metadata. Non-namespaced items, such as Puppet 3.x functions, will always be reported as near matches only.

lollipopman commented 1 year ago

Thanks for reviewing @smortex & @b4ldr bundle exec rake rubocop now passes clean on my pushed branch.