Skip to content

Remove mutation testing#440

Merged
samdark merged 3 commits intomasterfrom
copilot/remove-mutation-testing
Feb 20, 2026
Merged

Remove mutation testing#440
samdark merged 3 commits intomasterfrom
copilot/remove-mutation-testing

Conversation

Copy link
Contributor

Copilot AI commented Feb 20, 2026

  • Remove roave/infection-static-analysis-plugin from require-dev in composer.json
  • Remove infection/extension-installer from allow-plugins in composer.json
  • Remove infection.json.dist from paths-ignore in .github/workflows/build.yml
  • Remove infection.json.dist from paths-ignore in .github/workflows/static.yml
  • Remove infection.json.dist from paths-ignore in .github/workflows/composer-dependency-analyzer.yml
  • Add CHANGELOG entry for mutation testing removal
Original prompt

This section details on the original issue you should resolve

<issue_title>Remove mutation testing</issue_title>
<issue_description>### Proposed new feature or change

I think this is overhead for most projects.</issue_description>

Comments on the Issue (you are @copilot in this section)

@samdark I agree.

🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

Co-authored-by: samdark <47294+samdark@users.noreply.github.com>
Copilot AI changed the title [WIP] Remove mutation testing due to project overhead Remove mutation testing Feb 20, 2026
Copilot AI requested a review from samdark February 20, 2026 09:41
@samdark
Copy link
Member

samdark commented Feb 20, 2026

@copilot add a line for CHANGELOG.

Co-authored-by: samdark <47294+samdark@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Feb 20, 2026

@copilot add a line for CHANGELOG.

Added in 648bef1.

@samdark samdark marked this pull request as ready for review February 20, 2026 09:43
Copilot AI review requested due to automatic review settings February 20, 2026 09:43
Copy link

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

Removes mutation-testing tooling/configuration remnants from the Yii3 app template so the project no longer carries Infection-related dev dependencies or workflow trigger exclusions.

Changes:

  • Removed roave/infection-static-analysis-plugin from require-dev and infection/extension-installer from allow-plugins in composer.json.
  • Removed infection.json.dist from paths-ignore in GitHub Actions workflows.
  • Added a changelog entry noting mutation testing removal.

Reviewed changes

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

Show a summary per file
File Description
composer.json Drops Infection-related dev dependency and plugin allowance.
CHANGELOG.md Documents the change in the “under development” section.
.github/workflows/static.yml Stops ignoring infection.json.dist changes for static analysis workflow triggers.
.github/workflows/composer-dependency-analyzer.yml Stops ignoring infection.json.dist changes for dependency analyzer workflow triggers.
.github/workflows/build.yml Stops ignoring infection.json.dist changes for build workflow triggers.

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

@codecov
Copy link

codecov bot commented Feb 20, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 76.96%. Comparing base (faf21f4) to head (648bef1).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #440   +/-   ##
=========================================
  Coverage     76.96%   76.96%           
  Complexity       29       29           
=========================================
  Files            11       11           
  Lines           178      178           
=========================================
  Hits            137      137           
  Misses           41       41           

☔ View full report in Codecov by Sentry.
📢 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.

@samdark samdark merged commit da346dc into master Feb 20, 2026
24 checks passed
@samdark samdark deleted the copilot/remove-mutation-testing branch February 20, 2026 09:46
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.

Remove mutation testing

3 participants