revert e6a196feb4 #25

Merged
luke merged 1 commits from luke-patch-1 into main 2025-10-30 20:37:25 +00:00
Owner

revert chore(deps): update https://git.tainton.uk/actions/conventional-commits-check-action action to v1.3.0 (#23)

This PR contains the following updates:

Package Type Update Change
https://git.tainton.uk/actions/conventional-commits-check-action action minor v1.2.4 -> v1.3.0

Release Notes

actions/conventional-commits-check-action (https://git.tainton.uk/actions/conventional-commits-check-action)

v1.3.0

Compare Source

Features
  • deps: update conventional-pre-commit to 4.3.0 (b6d1665)

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 is behind base branch, or you tick the rebase/retry checkbox.

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


  • If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

Reviewed-on: #23
Reviewed-by: Luke Tainton luke@tainton.uk
Co-authored-by: renovate[bot] renovate-bot@git.tainton.uk
Co-committed-by: renovate[bot] renovate-bot@git.tainton.uk

revert chore(deps): update https://git.tainton.uk/actions/conventional-commits-check-action action to v1.3.0 (#23) This PR contains the following updates: | Package | Type | Update | Change | |---|---|---|---| | [https://git.tainton.uk/actions/conventional-commits-check-action](https://git.tainton.uk/actions/conventional-commits-check-action) | action | minor | `v1.2.4` -> `v1.3.0` | --- ### Release Notes <details> <summary>actions/conventional-commits-check-action (https://git.tainton.uk/actions/conventional-commits-check-action)</summary> ### [`v1.3.0`](https://git.tainton.uk/actions/conventional-commits-check-action/releases/tag/v1.3.0) [Compare Source](https://git.tainton.uk/actions/conventional-commits-check-action/compare/v1.2.4...v1.3.0) ##### Features - **deps:** update conventional-pre-commit to 4.3.0 ([b6d1665](https://git.tainton.uk/actions/conventional-commits-check-action/commit/b6d16651374e21a51c1c1113be63c775c21287fb)) </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 is behind base branch, 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:eyJjcmVhdGVkSW5WZXIiOiI0MS4xNjUuNSIsInVwZGF0ZWRJblZlciI6IjQxLjE2NS41IiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6WyJ0eXBlL2RlcGVuZGVuY2llcyJdfQ==--> Reviewed-on: https://git.tainton.uk/repos/docker-dnsmasq/pulls/23 Reviewed-by: Luke Tainton <luke@tainton.uk> Co-authored-by: renovate[bot] <renovate-bot@git.tainton.uk> Co-committed-by: renovate[bot] <renovate-bot@git.tainton.uk>
luke added 1 commit 2025-10-30 20:37:21 +00:00
revert e6a196feb4
Some checks failed
Validate PR Title / validate (pull_request) Failing after 2s
CI / ci (pull_request) Successful in 7s
8eddd75222
revert chore(deps): update https://git.tainton.uk/actions/conventional-commits-check-action action to v1.3.0 (#23)

This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [https://git.tainton.uk/actions/conventional-commits-check-action](https://git.tainton.uk/actions/conventional-commits-check-action) | action | minor | `v1.2.4` -> `v1.3.0` |

---

### Release Notes

<details>
<summary>actions/conventional-commits-check-action (https://git.tainton.uk/actions/conventional-commits-check-action)</summary>

### [`v1.3.0`](https://git.tainton.uk/actions/conventional-commits-check-action/releases/tag/v1.3.0)

[Compare Source](https://git.tainton.uk/actions/conventional-commits-check-action/compare/v1.2.4...v1.3.0)

##### Features

- **deps:** update conventional-pre-commit to 4.3.0 ([b6d1665](b6d1665137))

</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 is behind base branch, 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:eyJjcmVhdGVkSW5WZXIiOiI0MS4xNjUuNSIsInVwZGF0ZWRJblZlciI6IjQxLjE2NS41IiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6WyJ0eXBlL2RlcGVuZGVuY2llcyJdfQ==-->

Reviewed-on: #23
Reviewed-by: Luke Tainton <luke@tainton.uk>
Co-authored-by: renovate[bot] <renovate-bot@git.tainton.uk>
Co-committed-by: renovate[bot] <renovate-bot@git.tainton.uk>
luke merged commit 0292d3b27f into main 2025-10-30 20:37:25 +00:00
luke deleted branch luke-patch-1 2025-10-30 20:37:25 +00:00
Member

Decision: Reject
Change Score: 2/10

This change proposes downgrading the version of the conventional-commits-check-action used in the workflow. Downgrading actions is generally a bad idea unless there's a very specific and well-understood reason (e.g., a critical bug in the newer version that affects you, and that is not the case here). You're moving from v1.3.0 to v1.2.4, which likely includes bug fixes, improvements, and possibly even security patches that you would be missing out on. This is almost certainly not the right approach, and I strongly advise against it unless you have a concrete justification. Sticking to newer versions is almost always preferable, especially within the same major version, since you're likely to receive security patches and bug fixes.

If there is a problem that v1.3.0 causes for you, you should detail it in the commit message and the review request, so that it can be evaluated.

I am rejecting this change.

**Decision: Reject** **Change Score: 2/10** This change proposes downgrading the version of the `conventional-commits-check-action` used in the workflow. Downgrading actions is generally a bad idea unless there's a very specific and well-understood reason (e.g., a critical bug in the newer version that affects you, and that is not the case here). You're moving from v1.3.0 to v1.2.4, which likely includes bug fixes, improvements, and possibly even security patches that you would be missing out on. This is almost certainly not the right approach, and I strongly advise against it unless you have a concrete justification. Sticking to newer versions is almost always preferable, especially within the same major version, since you're likely to receive security patches and bug fixes. If there is a problem that v1.3.0 causes for you, you should detail it in the commit message and the review request, so that it can be evaluated. I am rejecting this change.
Sign in to join this conversation.
No Reviewers
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: repos/docker-dnsmasq#25