Abort if no change

[Lee-W]

• Abort cz commit it there is nothing to the commit
  • The underhook git commit follows cz-cli::src/commitizen/staging.js#L9. Although in cz-cli::src/cli/strategies/git-cz.js/#L51, they provide a --allow-empty, it seems not necessary to me. What ddo you think?

#45

[codecov-io]

Codecov Report

Merging #49 into master will decrease coverage by 4.94%.
The diff coverage is 100%.

@@ Coverage Diff @@## master #49 +/- ##
=========================================
- Coverage 93.05% 88.1% -4.95% 
=========================================
Files 24 24 Lines 547 555 +8 =========================================
- Hits 509 489 -20 - Misses 38 66 +28

Impacted Files	Coverage Δ
commitizen/git.py	100% <100%> (ø)	⬆️
commitizen/commands/commit.py	40.42% <100%> (-52.6%)	⬇️
commitizen/out.py	86.66% <0%> (-13.34%)	⬇️
commitizen/cz/base.py	95.83% <0%> (-4.17%)	⬇️

---

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 1f6ffb0...373ec25. Read the comment docs.

[woile]

Fine by me, if eventually someone needs --alow-empty we can consider it.

[alecgerona]

@woile I think the time has come. I think we need to respect the --allow-empty flag.

[woile]

Hahahah 😱
Please create a ticket describing the problem and proposed solution (sorry for the burocracy but otherwise we will forget 😅)