Print Git command if it failed

[capsuleman]

Description

If commit fails to execute, cli prints command so it can be copy / paste without executing again gitmoji-cli.
Updated the tests too.

Issue: #680

Tests

• All tests passed.

[carloscuesta]

Hey!

Thanks for opening a PR

Is there any difference between copying the command from the output of the error and using the arrow up key to replicate the same comand without having to manually copy&pasting it? IMHO using arrow key should be much faster, so I don't really see the point of this 🤔

[capsuleman]

Hey!

Thanks for opening a PR

Is there any difference between copying the command from the output of the error and using the arrow up key to replicate the same comand without having to manually copy&pasting it? IMHO using arrow key should be much faster, so I don't really see the point of this thinking

When I use gitmoji -c, I do not provide any other parameters and enter gitmoji and message in the cli. If I use arrow up key, I can not find again the parameters I filled previously.
If there is a quick way without copy/paste, I would be happy to hear it! :-)

[carloscuesta]

I think a screenshot/demo about this would be really helpful 😊

[capsuleman]

output.mp4

[carloscuesta]

Thanks for sharing! But this is the current behaviour right? I was asking for a demo of your change to see what is printed, thank you 🙏🏼

[capsuleman]

Oh sorry, I was just wondering if I was using the cli well. With the new version the command would be printed if it fail

simplescreenrecorder-2021-09-16_18.39.48.mp4

[carloscuesta]

Thanks for sharing the demo!

Currently, we have a way for specifying the default parameters through the cli. With this you can retry the command faster if it fails as prefills all the questions from the cli.

I'm not 100% if this is worth to be added into the cli, as it shows the underlying command and we should "abstract" that to the user 😅

[capsuleman]

Yes sure, but often I think about the gitmoji, title and message after having typed gitmoji -c...
Would you prefer having the command with default values printed? Is there a parameter to provide a gitmoji?

[codecov]

Codecov Report

Merging #681 (05f5b77) into master (d3ad2eb) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #681   +/-   ##
=======================================
  Coverage   92.13%   92.13%           
=======================================
  Files          26       26           
  Lines         229      229           
  Branches       49       49           
=======================================
  Hits          211      211           
  Misses         17       17           
  Partials        1        1           

Impacted Files	Coverage Δ
src/commands/commit/withClient/index.js	100.00% <ø> (ø)

---

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 d3ad2eb...05f5b77. Read the comment docs.

[carloscuesta]

Let's merge it 😊