Add a warning for resources outside of a class about puppet-lint

[rodjek]

Should it? Declaring resources outside the scope of a class or define is valid (and a trick I tend to use to great effect).

[vStone]

It might catch some people that placed their closing } too early.

How do you use them within your modules? Have an example? I'm always interested in new tricks ;)

[rodjek]

I don't use an ENC, so I have a file per node that looks roughly like this

@pxe::client { 'mynode.example.com':
    mac => '01:23:45:67:89:AB',
    os  => 'lenny',
}

node 'mynode.example.com' {
   ...
}

This allows me to keep my PXE client config with my node definition. Exported resources aren't an option here as we need the PXE config in place before the node has an OS, let alone run Puppet.

[vStone]

Still, that's not inside a module is it?

[llowder]

I've written a check that does this. https://github.com/llowder/puppet-lint-resource_outside_class-check

[rnelson0]

Thank you @llowder!