add --top-scope-variables cli argument, fix #228

[jcharaoui]

Summary

This adds the --top-scope-variables CLI option that sets PuppetLint.configuration.top_scope_variables.

Related Issues (if any)

#228

Checklist

• 🟢 Spec tests.
• 🟢 Acceptance tests.
• Manually verified.

[jcharaoui]

Removed an extra blank line.

[jcharaoui]

Not sure how to deal with this issue, though:

spec (ruby 3.3) / spec: lib/puppet-lint/optparser.rb#L20
Metrics/MethodLength: Method has too many lines. [111/108]

[LukasAud]

Not sure how to deal with this issue, though:

spec (ruby 3.3) / spec: lib/puppet-lint/optparser.rb#L20
Metrics/MethodLength: Method has too many lines. [111/108]

Manually adjusting the Metrics/MethodLength variable in the rubocop_todo.yml file should do the trick. Alternatively, you can run bundle exec rubocop --auto-gen-config to have Rubocop regenerate the file with the new exception.

[jcharaoui]

I fixed the comment and regenerated the Rubocop config for the line length issue.

[jcharaoui]

Fixed the explicit namespace failure. Specs should now pass. Sorry for the churn here.

[LukasAud]

LGTM