Skip to content

Add five more cases of superfluous actions and recommend their replacements#1937

Merged
woodruffw merged 3 commits intozizmorcore:mainfrom
agriyakhetarpal:more-superfluous-actions
Apr 23, 2026
Merged

Add five more cases of superfluous actions and recommend their replacements#1937
woodruffw merged 3 commits intozizmorcore:mainfrom
agriyakhetarpal:more-superfluous-actions

Conversation

@agriyakhetarpal
Copy link
Copy Markdown
Contributor

@agriyakhetarpal agriyakhetarpal commented Apr 22, 2026

Pre-submission checks

Please check these boxes:

  • Mandatory: This PR corresponds to an issue (if not, please create
    one first).

  • Having read the AI policy, I hereby disclose the use of an LLM or other
    AI coding assistant in the creation of this PR. PRs will not be rejected
    for using AI tools, but will be rejected for undisclosed use or
    use that violates the policy.

If a checkbox is not applicable, you can leave it unchecked.

Summary

In continuation of my previous contribution via #1873, I discovered that some actions across the projects I am involved in are superfluous and can be replaced with simpler commands. This PR adds them to the list. All of these have more than a hundred uses at this time, as noted in the table below.

Action Why? Usage metrics
actions-ecosystem/action-add-labels Can be replaced with gh issue or gh pr commands Search results (3.6K hits)
actions-ecosystem/action-remove-labels Can be replaced with gh issue or gh pr commands Search results (2.3K hits)
sergeysova/jq-action It's literally just jq! It's installed by default on GHA Linux runners. However, jq needs to be installed on macOS runners (via Homebrew or other package managers), from the last time I remember trying it. Search results (636 hits, and this is pretty surprising to me! 🙈)
stefanzweifel/git-auto-commit-action Commonly used to add files and commit them to a PR or a branch. I've noted this with a pedantic persona and a low-confidence score: there are some features, such as the dirty check (git status --porcelain plus an early exit), committer identity config, and push setup, etc., that require enough boilerplate to justify their existence. Search results (44.5K hits)
EndBug/add-and-commit Same use case as stefanzweifel. It also exposes outputs like commit_long_sha and pushed, which downstream steps may consume — those need git rev-parse HEAD, and so on to replicate. Again, I believe this is all doable with git, but feel that there are enough moving parts for Pedantic/Low. Search results (21.6K hits)

Thank you :D

Copy link
Copy Markdown
Member

@woodruffw woodruffw left a comment

Choose a reason for hiding this comment

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

Thanks @agriyakhetarpal! These persona/confidence levels seem reasonable to me on first blush.

@woodruffw woodruffw enabled auto-merge (squash) April 23, 2026 02:30
@woodruffw woodruffw added the enhancement New feature or request label Apr 23, 2026
@woodruffw woodruffw merged commit 1a1bf15 into zizmorcore:main Apr 23, 2026
16 checks passed
@agriyakhetarpal agriyakhetarpal deleted the more-superfluous-actions branch April 23, 2026 04:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants