feat!: remove auto-snapshot on dirty tag

[caarlos0]

closes #380

[codecov]

Codecov Report

Merging #382 (f86b042) into master (b508e2e) will decrease coverage by 1.09%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #382      +/-   ##
==========================================
- Coverage   66.14%   65.04%   -1.10%     
==========================================
  Files           4        3       -1     
  Lines         127      103      -24     
  Branches       23       17       -6     
==========================================
- Hits           84       67      -17     
+ Misses         25       23       -2     
+ Partials       18       13       -5     

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

[crazy-max]

Sorry didn't have time to review this 😣 but lgtm

[caarlos0]

No worries ❤️

[meain]

Can I get the previous behavior of using snapshot if dirty and version of tagged by adding --snapshot to the args or will I have to separately handle them in each case?

[caarlos0]

there's a --auto-snapshot flag that enables it if the tree is dirty, you can either use that + a step to make the tree dirty on purpose that only runs when HEAD is not a tag, split the workflow into 2 (one for tags that never snapshots, one for commits that always snapshots), or handle with if statements.

FWIW: I do something similar here https://github.com/caarlos0/goreleaser-pro-split-merge-example/blob/99553a5ca10b82335ae995083210957bb574ff8a/.github/workflows/nightly.yml#L23 for nightly on workflow_dispatch.