Skip to content

Conversation

@biamandei
Copy link
Collaborator

@biamandei biamandei commented Jan 9, 2023

This fixes the following problems in the following types:

  • namespace_bindings type: remove : from the list of valid characters in the composite namevars
  • namespace_bindings type: fix missing matching regexp group for the server composite namevar option
  • shared_library type: remove : from the list of valid characters in the composite namevars
  • shared_library type: fix missing matching regexp group for the server composite namevar option
  • shared_library type: fix inconsistently named variable from node to node_name
  • websphere_variable type: remove : from the list of valid characters in the composite namevars
  • websphere_variable type: fix missing matching regexp group for the server composite namevar option
  • websphere_variable type: remove cell from the list of requirements when scope is node

Fix requirement for node_name in cell scope context
Fix invalid variable reference to node: should be node_name
@biamandei biamandei requested a review from a team as a code owner January 9, 2023 12:21
@puppet-community-rangefinder
Copy link

websphere_namespace_binding is a type

that may have no external impact to Forge modules.

websphere_shared_library is a type

that may have no external impact to Forge modules.

websphere_variable is a type

that may have no external impact to Forge modules.

This module is declared in 0 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.

@CLAassistant
Copy link

CLAassistant commented Apr 19, 2023

CLA assistant check
All committers have signed the CLA.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants