Skip to content

fix(ci): pcov を setup-php に統一して php8.5-pcov の apt 取得失敗を回避#6850

Merged
ttokoro20240902 merged 1 commit into
4.4from
fix/coverage-pcov-8.5
Jun 22, 2026
Merged

fix(ci): pcov を setup-php に統一して php8.5-pcov の apt 取得失敗を回避#6850
ttokoro20240902 merged 1 commit into
4.4from
fix/coverage-pcov-8.5

Conversation

@ttokoro20240902

@ttokoro20240902 ttokoro20240902 commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

概要

coverage / PHPUnit (8.5, pgsql) ジョブが以下で失敗するようになったため修正します。

sudo apt-get install -y php8.5-pcov
E: Unable to locate package php8.5-pcov
##[error]Process completed with exit code 100

原因

  • coverage.yml は pcov を apt(Ondřej PPA)から OS パッケージとして導入していた(2021年の導入時からの方式。apt-fast→apt-get と変遷)。
  • php8.5-pcov が apt で取得できなくなり、apt-get install が "Unable to locate package" で失敗。
  • 2026-06-11 時点では成功していたため、ワークフロー側の変更ではなく apt/PPA 側の提供状況変化による外部要因の回帰

修正

pcov のインストールを apt から shivammathur/setup-phpcoverage: pcov 入力に統一し、PPA の在庫状況に依存しないようにします。すぐ隣で同アクションを PHP 導入に使っているため、自然に寄せられます。

         with:
           php-version: ${{ matrix.php }}
           extensions: redis
-
-      - name: Setup pcov
-        run: |
-          sudo apt-get install -y php${{ matrix.php }}-pcov
-          sudo phpenmod -s cli pcov
+          coverage: pcov

備考

  • codeception ジョブにも同様の apt 経由 pcov 導入(php8.4-pcov)がありますが、当該ジョブは if: false で無効化済みのため本PRの対象外としています。
  • 動作確認: 本PRの CI で coverage / PHPUnit (8.5, pgsql)Setup PHP ステップが成功し pcov が有効化されることを確認します。

🤖 Generated with Claude Code

Summary by CodeRabbit

リリースノート

  • Chores
    • テスト実行環境のセットアップを効率化しました。PCOV(コードカバレッジツール)の有効化プロセスが簡潔になり、テストの実行がより効率的になりました。

php8.5-pcov が apt(Ondřej PPA)で取得できなくなり coverage ジョブが
"Unable to locate package php8.5-pcov" で失敗するようになった。
pcov のインストールを apt から setup-php の coverage 入力に寄せ、
PPA の在庫状況に依存しないようにする。
(codeception ジョブは if:false で無効のため対象外)

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Jun 22, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 3b34c375-e0d4-44f6-ba01-25adef8bbe1f

📥 Commits

Reviewing files that changed from the base of the PR and between 5777408 and 639bd12.

📒 Files selected for processing (1)
  • .github/workflows/coverage.yml

📝 Walkthrough

Walkthrough

.github/workflows/coverage.ymlphpunit ジョブで、PCOV の有効化方法を変更。shivammathur/setup-php アクションに coverage: pcov を追加し、従来の apt-get installphpenmod による手動インストールステップを削除。

Changes

PCOV セットアップ方法の移行

Layer / File(s) Summary
PCOV セットアップをアクション設定へ移行
.github/workflows/coverage.yml
Setup PHP ステップに coverage: pcov オプションを追加し、apt-get install php*-pcovphpenmod pcov を実行していた Setup pcov ステップを削除。

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐇 うさぎはぴょんと跳ねながら言う
apt-get を捨てて、設定ひとつでOK!
coverage: pcov とそっと書けば
余分なステップはもう要らない
シンプルが一番、ぴょん! 🎉

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed プルリクエストのタイトルは、CI ワークフローで pcov インストール方法を setup-php アクションに統一し、apt 経由の取得失敗を回避するという主要な変更内容を明確かつ簡潔に要約しています。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/coverage-pcov-8.5

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@nanasess nanasess left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@codecov

codecov Bot commented Jun 22, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.93%. Comparing base (5777408) to head (639bd12).

Additional details and impacted files
@@            Coverage Diff             @@
##              4.4    #6850      +/-   ##
==========================================
+ Coverage   74.88%   74.93%   +0.04%     
==========================================
  Files         463      463              
  Lines       24029    24029              
==========================================
+ Hits        17995    18005      +10     
+ Misses       6034     6024      -10     
Flag Coverage Δ
Unit 74.93% <ø> (+0.04%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ttokoro20240902 ttokoro20240902 merged commit 14fd473 into 4.4 Jun 22, 2026
352 checks passed
@ttokoro20240902 ttokoro20240902 deleted the fix/coverage-pcov-8.5 branch June 22, 2026 23: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.

3 participants