Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Drop support for Laravel < 10 #238

Merged
merged 7 commits into from
Mar 19, 2024
Merged

Conversation

mfn
Copy link
Contributor

@mfn mfn commented Mar 18, 2024

Description

Similar to #229 but only concentrate o the part to drop support for unsupported software.

This also means we drop PHP < 8.1, as 8.1 is the minimum, see https://laravel.com/docs/11.x/releases#support-policy , as Laravel 9 is EOL since a month:
image

Also cleaned up a bit:

  • Bump other PHP versions to their latest stable release (8.3) for GHA workflows.
  • Removed phpdotenv, not needed 🤷🏼
  • Removed yoast/phpunit-polyfills, not needed 🤷🏼

Checklist:

  • I've added tests for my changes or tests are not applicable
  • I've changed documentations or changes are not required
  • I've added my changes to CHANGELOG.md

mfn added 7 commits March 18, 2024 13:41
This also means we drop PHP < 8.1, as 8.1 is the minimum.

Also take the opportunity to bump other PHP versions to their
latest stable release (8.3) for GHA workflows.

Also removed the explicit dotenv references, don't seem to be necessary.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

There is really a lot of stuff not necessary anymore!

Comment on lines -54 to -55
"vlucas/phpdotenv": "^5.2.0",
"yoast/phpunit-polyfills": "^2.0.0"
Copy link
Contributor Author

@mfn mfn Mar 18, 2024

Choose a reason for hiding this comment

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

Turns out these two dev deps were not needed anymore 🤷🏼

"orchestra/testbench": "^4.18|^5.8|^6.3|^7|^8|^9|9.x-dev",
"illuminate/console": "^10|^11",
"illuminate/routing": "^10|^11",
"orchestra/testbench": "^8|^9",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Removed 9.x-dev for testbench, was only necessary before the Laravel 11 release

Comment on lines +73 to +76
-
message: "#^Method PHPOpenSourceSaver\\\\JWTAuth\\\\Test\\\\Stubs\\\\LaravelUserStub\\:\\:getAuthPasswordName\\(\\) should return string but return statement is missing\\.$#"
count: 1
path: tests/Stubs/LaravelUserStub.php
Copy link
Contributor Author

Choose a reason for hiding this comment

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

phpstan with latest PHP 8.3 produced this; as with all errors from Stubs, added it here (i.e. there are already others in the neon file here too).

@mfn
Copy link
Contributor Author

mfn commented Mar 18, 2024

@Messhias before merging this, please see #236 (comment) : we should make first a release with Carbon 3 support

@Messhias
Copy link
Collaborator

@Messhias before merging this, please see #236 (comment) : we should make first a release with Carbon 3 support

No problem, I'll release everything together, I'll see this one today in the evening and merge it, only on the weekend I release something in packist.

@mfn
Copy link
Contributor Author

mfn commented Mar 18, 2024

No problem, I'll release everything together,

Agree, makes sense. Carbon 3 is only usable with Laravel 11 anyway 👍🏼

@Messhias
Copy link
Collaborator

No problem, I'll release everything together,

Agree, makes sense. Carbon 3 is only usable with Laravel 11 anyway 👍🏼

No problem, I already released it anyway because there might be other people who want to use this package and can't do it because of the version. Since it was already tested I didn't mind releasing it.

Today I'll review this one as well.

@Messhias Messhias merged commit de4b752 into PHP-Open-Source-Saver:main Mar 19, 2024
12 checks passed
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.

2 participants