New CLI commands

[neomilium]

Main goal is to be able to execute arbitrary commands on each repositories.

This PR contains a draft for new commands : reset, execute and push

• clone: clone repo if not already done
• execute: run arbitrary command
• reset: hard-reset repositories
• push: push commits

Any help to define expected behaviors is welcome.

[neomilium]

Please note that this PR is on top of #206

[codecov]

Codecov Report

Merging #244 (0c9838c) into master (5b33600) will increase coverage by 2.37%.
The diff coverage is 96.42%.

@@            Coverage Diff             @@
##           master     #244      +/-   ##
==========================================
+ Coverage   84.76%   87.14%   +2.37%     
==========================================
  Files          30       30              
  Lines         906     1011     +105     
==========================================
+ Hits          768      881     +113     
+ Misses        138      130       -8     

Impacted Files	Coverage Δ
lib/modulesync/git_service.rb	70.58% <50.00%> (+16.24%)	⬆️
lib/modulesync/source_code.rb	97.50% <50.00%> (-2.50%)	⬇️
lib/modulesync/git_service/factory.rb	92.30% <80.00%> (+1.39%)	⬆️
lib/modulesync/git_service/base.rb	90.47% <90.90%> (+0.47%)	⬆️
lib/modulesync.rb	98.57% <97.67%> (-0.42%)	⬇️
features/step_definitions/git_steps.rb	96.15% <100.00%> (+0.32%)	⬆️
lib/modulesync/cli.rb	100.00% <100.00%> (ø)
lib/modulesync/git_service/gitlab.rb	100.00% <100.00%> (ø)
lib/modulesync/hook.rb	100.00% <100.00%> (ø)
lib/modulesync/renderer.rb	100.00% <100.00%> (ø)
... and 3 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3c9e406...0c9838c. Read the comment docs.

[ekohl]

I would still love to see the execute command. It would still help me a lot.

[neomilium]

Code rebased, contributions are welcome, dev is stalled on this ;-)

[bastelfreak]

@neomilium do you still consider this a draft? can you please take a look at the rubocop issues?

[neomilium]

@bastelfreak Yes, its still a draft, it requires to be used and feedbacks are needed.

Rubocop issues should be fixed.

[neomilium]

One command is missing to the set to be really usable: pull-request or merge-request…

[ekohl]

I think this addresses all my concerns. It's still a large patch and I could have missed things, but I'm going to trust all the tests you added. Let's try this out since I'm excited to replace my bash for loops with msync exec, see how that feels. Fun fact: I actually prefer doing msync exec git pull over trusting msync. Feels like I get more control. Certainly something I had desired for a long time.

[ekohl]

Now that it's been released I'm really enjoying msync exec, but the one thing that bugs me is that it switches to the default branch. I had expected it to be a short hand for for module in modules/*/* ; do ( cd $i && $command ) ; done. Is this intentional or a bug?

[ekohl]

So I opened #251 for further discussion.