Skip to content

feat: allow for clean tag pushing#364

Merged
stefanzweifel merged 7 commits intostefanzweifel:masterfrom
zmynx:master
Apr 19, 2025
Merged

feat: allow for clean tag pushing#364
stefanzweifel merged 7 commits intostefanzweifel:masterfrom
zmynx:master

Conversation

@zMynxx
Copy link
Copy Markdown
Contributor

@zMynxx zMynxx commented Mar 23, 2025

This PR contains the code change to allow this action to be invoked for performing a clean tag & push.
By clean I mean to commits are added, and no file changes are staged.
It is meant to be used in CD systems, where we wish to alter the tag, without allowing automation entities to modify the source code (anti-pattern). This action can now be used a the de-facto standard for perform git operations in a pipeline / automations.

@zMynxx
Copy link
Copy Markdown
Contributor Author

zMynxx commented Mar 23, 2025

Follow up for my initial request #360 (reply in thread)

@stefanzweifel
Copy link
Copy Markdown
Owner

Thanks for the taking the time to create a PR for this. I will take a closer look in the coming days (currently sick and not on my computer).

At first glance, I think looks good. I will definitely write a few Bats tests for this, so future other releases won't break this functionality.

Copy link
Copy Markdown
Owner

@stefanzweifel stefanzweifel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry for my late reply here. Life got in the way.

Looks good to me, I will merge this now but will push some changes in a separate PR. As you've created this PR from a fork in an organisation, I can't push my changes to the PR. (Not your faults, this is a long standing issue of GitHub itself: https://github.com/orgs/community/discussions/5634)

One change is renaming the option from git_tag_only to create_git_tag_only. I've also added some tests.

@stefanzweifel stefanzweifel merged commit 11a6e5f into stefanzweifel:master Apr 19, 2025
2 checks passed
crookm pushed a commit to interflare/orleans-marten that referenced this pull request Mar 23, 2026
…tion to v5.2.0 (#22)

This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [https://enduringtech.dev/actions/commit-action](https://enduringtech.dev/actions/commit-action) | action | minor | `v5.1.0` -> `v5.2.0` |

---

### Release Notes

<details>
<summary>actions/commit-action (https://enduringtech.dev/actions/commit-action)</summary>

### [`v5.2.0`](https://enduringtech.dev/actions/commit-action/blob/HEAD/CHANGELOG.md#v520---2025-04-19)

[Compare Source](https://enduringtech.dev/actions/commit-action/compare/v5.1.0...v5.2.0)

##### Added

-   Add `create_git_tag_only` option to skip commiting and always create a git-tag. ([#&#8203;364](stefanzweifel/git-auto-commit-action#364)) [@&#8203;zMynxx](https://github.com/@&#8203;zMynxx)
-   Add Test for `create_git_tag_only` feature ([#&#8203;367](stefanzweifel/git-auto-commit-action#367)) [@&#8203;stefanzweifel](https://github.com/@&#8203;stefanzweifel)

##### Fixed

-   docs: Update README.md per [#&#8203;354](https://github.com/actions/commit-action/issues/354) ([#&#8203;361](stefanzweifel/git-auto-commit-action#361)) [@&#8203;rasa](https://github.com/@&#8203;rasa)

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box

---

This PR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzOS4yMjcuMiIsInVwZGF0ZWRJblZlciI6IjM5LjIyNy4yIiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6W119-->

Reviewed-on: https://enduringtech.dev/interflare/orleans-marten/pulls/22
Reviewed-by: Matt <mattlcrook@icloud.com>
Co-authored-by: Renovate Bot <renovate.svc@enduringtech.dev>
Co-committed-by: Renovate Bot <renovate.svc@enduringtech.dev>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants