Skip to content

chore: simplify actions-maintenance wrapper triggers and job name - #82

Merged
krukonshedul merged 2 commits into
masterfrom
chore/actions-maintenance-wrapper-cleanup
Sep 16, 2026
Merged

krukonshedul merged 2 commits into
masterfrom
chore/actions-maintenance-wrapper-cleanup

Conversation

@krukonshedul

@krukonshedul krukonshedul commented Sep 16, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Drop the explicit pull_request: types: list — it duplicated GitHub's own default (opened, synchronize, reopened), so removing it changes nothing at runtime.
  • Rename the wrapper's job id from actions-maintenance to phase (displayed as Phase), so check names read Actions Maintenance / Phase / <role> instead of the redundant actions-maintenance / actions-maintenance / <role>.
  • If this repo has a push trigger on the wrapper, align its presence with whether this repo is a Source in actions-manifest.yaml (owns a registered internal action) — added where missing, removed where present but not warranted.

Part of an org-wide cleanup of the actions-maintenance.yaml wrapper across ~355 repos.

  • Also synced trailing # vX version comments on uses: pins with the manifest's source_ref where they'd drifted from a prior SHA-only bump (SHA itself was already correct; comment-only follow-up commit)

Test plan

  • actionlint passes on the changed workflow file
  • Diff reviewed for correctness against the reusable workflow's expected inputs

🤖 Generated with Claude Code

krukonshedul and others added 2 commits September 16, 2026 11:45
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

@dyl-sv dyl-sv left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Automated review: change is limited to the actions-maintenance workflow rename/trigger cleanup and/or SHA-pinned comment corrections (verified no dependency SHAs changed). Approving.

@krukonshedul
krukonshedul merged commit 391a084 into master Sep 16, 2026
6 checks passed
@krukonshedul
krukonshedul deleted the chore/actions-maintenance-wrapper-cleanup branch September 16, 2026 16:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants