Skip to content

ci(dependabot): npm の [skip ci] prefix を削除し全てのCIを実行する#371

Merged
ROhta merged 1 commit into
mainfrom
worktree-dependabot-remove-skip-ci
May 23, 2026
Merged

ci(dependabot): npm の [skip ci] prefix を削除し全てのCIを実行する#371
ROhta merged 1 commit into
mainfrom
worktree-dependabot-remove-skip-ci

Conversation

@ROhta

@ROhta ROhta commented May 23, 2026

Copy link
Copy Markdown
Owner

期待する挙動・状態

  • npm dependabot のPR でも、 すべての CI / 外部ツールを通常通り実行する
  • コミットメッセージは dependabot デフォルト (例: `Bump xxx from 1.0.0 to 1.1.0`)

必要な依存関係

  • なし

確認済み項目

見てほしいところ

  • PRのLabelsの設定は適切か
  • github-actions の `check ci status` プレフィックスも統一(削除)するか、 現状維持か
  • dependabot がデフォルトのコミットメッセージで動作することを次回 dependabot 起動時 (npm: daily 10:00 JST) に確認

背景

[skip ci] プレフィックスでCIをスキップする方針から、 dependabot のPRでもCIを全て走らせる 方針に転換。 理由:

  • GitHub Actions の [skip ci]pull_request の特殊 types (review_requested / ready_for_review / reopened) における挙動が公式に明示されておらず信頼性に欠ける
  • 外部ツール (CodeQL / DeepScan / Snyk) は GitHub Actions の管轄外で [skip ci] の影響を受けない
  • 部分的にしかスキップできないなら、 一貫してすべて実行する方が混乱がない

変更内容

ファイル 変更
.github/dependabot.yml npm エコシステムの commit-message セクション (prefix [skip ci] + include scope) を削除

github-actions エコシステムは既存のコミット履歴 (check ci status(deps): bump ...) との連続性のため変更なし。

dependabotのPRでもCIを全て走らせる方針に変更したため、npmエコシステムの
commit-messageセクションを削除しデフォルトのコミットメッセージに戻す。

GitHub Actionsの[skip ci]はpull_requestの特殊types (review_requested等)
での挙動が公式に明示されておらず信頼性に欠ける点、外部ツール
(CodeQL/Snyk/DeepScan)は管轄外で常に走る点を踏まえ、CIをスキップする
仕組み自体を廃止する。

github-actionsエコシステムの "check ci status" プレフィックスは
既存のコミット履歴との連続性のため維持。
Copilot AI review requested due to automatic review settings May 23, 2026 02:11
@ROhta

ROhta commented May 23, 2026

Copy link
Copy Markdown
Owner Author

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

Copilot AI 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.

Pull request overview

npm の Dependabot PR でも [skip ci] による CI スキップを行わず、通常どおり全ての CI / 外部ツールが実行される状態に戻すための変更です。リポジトリの依存関係更新 PR をより一貫して検証できるようにする意図に沿っています。

Changes:

  • .github/dependabot.yml の npm エコシステム設定から commit-message.prefix: "[skip ci]"(および include: "scope")を削除
  • dependabot のデフォルトコミットメッセージ(例: Bump ... from ... to ...)に戻す

@ROhta
ROhta enabled auto-merge (squash) May 23, 2026 02:14
@ROhta
ROhta disabled auto-merge May 23, 2026 02:14
@ROhta
ROhta merged commit 7f5de8d into main May 23, 2026
9 checks passed
@ROhta
ROhta deleted the worktree-dependabot-remove-skip-ci branch May 23, 2026 02:14
@ROhta ROhta self-assigned this May 23, 2026
@ROhta ROhta added the bug-3 改善 割れ窓で直す label May 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug-3 改善 割れ窓で直す

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants