Skip to content

Updates missing .phpunit.cache dir for newer laravel#4873

Merged
dooleydevin merged 1 commit into
github:mainfrom
lissy93-forks:updates-laravel-phpcache-dir
Jul 23, 2026
Merged

Updates missing .phpunit.cache dir for newer laravel#4873
dooleydevin merged 1 commit into
github:mainfrom
lissy93-forks:updates-laravel-phpcache-dir

Conversation

@lissy93

@lissy93 lissy93 commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Link to the application or project's homepage

https://laravel.com/

Reasons for making this change

laravel moved the default location for test results and coverage cache back in 2023, to just .phpunit.cache.
The old .phpunit.result.cache is still useful for projects using PHPUnit 9 or below, so I didn't touch that.

Links to documentation supporting these rule changes

Official Laravel .gitignore: https://github.com/laravel/laravel/blob/master/.gitignore#L11
(added in laravel/laravel#6052, during the PHPUnit 10 upgrade)

Merge and Approval Steps

Copilot AI review requested due to automatic review settings June 18, 2026 10:20
@lissy93
lissy93 requested a review from a team as a code owner June 18, 2026 10:20

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

Pull request overview

Updates the Laravel.gitignore template to match newer Laravel/PHPUnit defaults by ignoring the PHPUnit cache directory introduced as the default location for test cache data.

Changes:

  • Add /.phpunit.cache to ignore PHPUnit 10+ cache directory while retaining .phpunit.result.cache for older PHPUnit versions.

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

@lissy93

lissy93 commented Jul 23, 2026

Copy link
Copy Markdown
Contributor Author

Hey @dooleydevin any chance for a review on this one? 🤗

@dooleydevin dooleydevin left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM! Thanks for the contribution!

@dooleydevin
dooleydevin merged commit 57286c3 into github:main Jul 23, 2026
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