Skip to content

chore: remove dependency-update.yml in favour of Renovate#24

Merged
toshi0806 merged 1 commit into
mainfrom
remove-dependency-update-workflow
May 23, 2026
Merged

chore: remove dependency-update.yml in favour of Renovate#24
toshi0806 merged 1 commit into
mainfrom
remove-dependency-update-workflow

Conversation

@toshi0806
Copy link
Copy Markdown
Member

Summary

  • 共有 reusable workflow `dependency-update.yml` を撤去
  • README から該当の節を削除し、依存関係更新は Renovate に集約された旨を追記

Background

このワークフローは呼び出し元の `secrets.GITHUB_TOKEN` で PR を作成しており、GitHub Actions の再帰防止仕様 により後続の `pull_request` イベントが起動しません。結果として呼び出し側リポジトリ(smkwlab/tenbin_dns, tdig, tenbin_cache, elixir_dnstap, tenbin_ex)で必須チェック `ci / Code Quality` が走らず BLOCKED 状態が続いていました。

呼び出し側 5 リポジトリは Renovate に移行済み(各 repo の `renovate.json` を参照)。Renovate は GitHub App として動作するため再帰制限を受けません。

Cross-org usage check

`gh search code 'dependency-update.yml' org:smkwlab` でこの workflow を参照するファイルは smkwlab/.github 自身のみ。撤去しても他リポジトリへの影響はありません。

Follow-up

  • 一時的に作成していた GitHub App `smkwlab-dep-updater` と関連 Org Secret (`DEPENDENCY_APP_ID` / `DEPENDENCY_APP_PRIVATE_KEY`) も後ほど撤去予定(Renovate 移行で不要)

The reusable Elixir dependency-update workflow created PRs with the
calling repository's `GITHUB_TOKEN`, which suppresses subsequent
`pull_request` workflow runs and left required CI checks unsatisfied
(BLOCKED state across smkwlab/tenbin_dns, tdig, tenbin_cache,
elixir_dnstap, and tenbin_ex).

Those repositories have been migrated to Renovate (renovate.json per
repo), which runs as a GitHub App and is not subject to the token
recursion restriction. A search of the smkwlab org confirms no other
repository still uses this reusable workflow, so remove the file and
its README entry.
Copilot AI review requested due to automatic review settings May 23, 2026 07:53
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR removes the organization’s reusable dependency-update.yml workflow and updates documentation to reflect that dependency updates are now handled exclusively by Renovate (as configured per-repo via renovate.json). This aligns with the stated background that PRs created via secrets.GITHUB_TOKEN don’t trigger downstream pull_request checks due to GitHub Actions recursion prevention, leaving required checks blocked.

Changes:

  • Delete the reusable workflow .github/workflows/dependency-update.yml.
  • Remove dependency-update.yml from the README workflow list and delete its documentation section.
  • Add a README note stating dependency updates are centralized to Renovate Bot.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
README.md Removes dependency-update.yml references and documents the move to Renovate.
.github/workflows/dependency-update.yml Deletes the reusable dependency update workflow entirely.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@toshi0806 toshi0806 merged commit cca892b into main May 23, 2026
2 checks passed
@toshi0806 toshi0806 deleted the remove-dependency-update-workflow branch May 23, 2026 07:59
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