(PDK-985) Split validation targets into chunks of 1000

[rodjek]

In order to work around argument list length limits.

[coveralls]

Coverage increased (+0.04%) to 92.895% when pulling b71286b on rodjek:pdk-985 into 455f488 on puppetlabs:master.

[scotje]

@rodjek is this still WIP?

[rodjek]

@scotje Not any more :)

[scotje]

There seem to be some issues with how the new code interacts with the multispinner:

$ lpdk validate --parallel
pdk (INFO): Running all available validators...
pdk (INFO): Using Ruby 2.4.4
pdk (INFO): Using Puppet 5.5.1
┌ [✔] Validating module using 3 threads        ┌
└──[✔] Checking metadata syntax (metadata.json ┌ ─ks/*.json).
└──[/] Checking module metadata style (metadata├──on).
└──[✔] Checking module metadata style (metadata├──on).
└──[✔] Checking task metadata style (tasks/*.json).
[✔] Checking Puppet manifest syntax (**/**.pp).
[✔] Checking Puppet manifest style (**/*.pp).
convention: rubocop: spec/unit/puppet/provider/testprovider/testprovider_spec.rb:4:1: Style/Documentation: Missing top-level module documentation comment.

vs master:

pdk (INFO): Running all available validators...
pdk (INFO): Using Ruby 2.4.4
pdk (INFO): Using Puppet 5.5.1
┌ [✖] Validating module using 3 threads
├──[✖] Checking Ruby code style (**/**.rb).
├──[✔] Checking metadata syntax (metadata.json tasks/*.json).
├──[✔] Checking Puppet manifest syntax (**/**.pp).
├──[✔] Checking module metadata style (metadata.json).
├──[✔] Checking task metadata style (tasks/*.json).
└──[✔] Checking Puppet manifest style (**/*.pp).
convention: rubocop: spec/unit/puppet/provider/testprovider/testprovider_spec.rb:4:1: Style/Documentation: Missing top-level module documentation comment.

I'm going to see if I can figure out what is going on.

[bmjen]

https://tickets.puppetlabs.com/browse/PDK-1032 Was created to track and fix the multi-spinner display.