fix rubocop removing trailing commas from multi-line declarations

[jhoblitt]

No description provided.

[codecov]

Codecov Report

Base: 1.63% // Head: 1.63% // No change to project coverage 👍

Coverage data is based on head (fe2563f) compared to base (53011ea).
Patch coverage: 0.00% of modified lines in pull request are covered.

❗ Current head fe2563f differs from pull request most recent head 43eb0cb. Consider uploading reports for the commit 43eb0cb to get more accurate results

Additional details and impacted files

@@          Coverage Diff           @@
##           master     #86   +/-   ##
======================================
  Coverage    1.63%   1.63%           
======================================
  Files           2       2           
  Lines          61      61           
======================================
  Hits            1       1           
  Misses         60      60           

Impacted Files	Coverage Δ
lib/voxpupuli/test/facts.rb	2.50% <0.00%> (ø)
lib/voxpupuli/test/rake.rb	0.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

[jhoblitt]

Rebased after the merge of #85.

[jhoblitt]

rubocop in this repo is now successfully eating its own dog food...

[bastelfreak]

before we merge this I think ot makes sense to have an overview of the impact. I am afraid we need to update all modules after this change is released

[jhoblitt]

Ya. I've been thinking about how to incorporate rubocop into modulesync. I'm also thinking that we should be pinning the version in modules in order to prevent a massive simultaneous update in the future.

[ekohl]

I'm strongly in favor of this, but it will cause changes in most modules so I'd suggest it's a major version bump. We do plan one anyway for dropping Ruby < 2.7, so perhaps we should align those.

[jhoblitt]

I'm wondering if the right approach is to do a module sync that adds a branch name to every pupmod that pins the file to the way it is now. Then we can merge this PR and an related PR to modulesync_config that bumps the branch ref. That will put off needing to modify individual mods until the next modsync run.

[ekohl]

We aim to make the module also usable outside of Vox Pupuli, so we can't control it. It must be a major version IMHO.