Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

test: merge tests to test multiple PHP and Laravel versions #215

Closed
wants to merge 12 commits into from

Conversation

pataar
Copy link

@pataar pataar commented Feb 10, 2024

I merged the tests to test combinations of Laravel 9, 10 and PHP 8.1 to 8.3.

Also removed the composer.lock file as packages shouldn't have one.

@pataar pataar changed the title test: add php 8.3 tests test: merge tests to test multiple php and laravel versions Feb 10, 2024
@pataar pataar changed the title test: merge tests to test multiple php and laravel versions test: merge tests to test multiple PHP and Laravel versions Feb 10, 2024
@hisorange
Copy link
Owner

Wow! Nice job, let me read this through to get a full picture, this GH action flow was setup long long ago, and for sure it was messy, so this is hugely appreciated!

@pataar
Copy link
Author

pataar commented Jan 27, 2025

@hisorange I merged these changes with required changes for PHP 8.4 in #230. So I'm closing this one.

@pataar pataar closed this Jan 27, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants