puppet-lint icon indicating copy to clipboard operation
puppet-lint copied to clipboard

Update the plugin puppet_url_without_modules

Open rnelson0 opened this issue 8 years ago • 2 comments

The current check looks for the modules endpoint, but users can create and maintain their own endpoints. The plugin should be renamed and check for a valid endpoint name AND a subsequent path of the correct format. Notes from #561:

  • puppet:///modules/modulename/file could be tested for with puppet:///modules/\w+/
  • puppet:///othermount/file and puppet:///othermount/path/file are both valid and would match ^puppet:///\w+/\w+
  • puppet:///modules/filename is not valid and could be tested for with puppet:///modules/\w+$

If that seems reasonable, I will open a new ticket to track that change and link to it in the OP.

rnelson0 avatar Nov 29 '16 23:11 rnelson0

From February Triage: This is not called out in the style guide and should be disabled and moved to a community plugin. It should be disabled in 3.0.0.

rnelson0 avatar Feb 23 '17 22:02 rnelson0

+1

JonathonAnderson avatar Oct 21 '21 16:10 JonathonAnderson