Skip to content

[10.x] Remove PHP 8.1 Check for including Enums in Tests - #48415

Merged
taylorotwell merged 1 commit into
laravel:10.xfrom
Jubeki:remove-php81-check
Sep 15, 2023
Merged

[10.x] Remove PHP 8.1 Check for including Enums in Tests#48415
taylorotwell merged 1 commit into
laravel:10.xfrom
Jubeki:remove-php81-check

Conversation

@Jubeki

@Jubeki Jubeki commented Sep 15, 2023

Copy link
Copy Markdown
Contributor
  • Some Tests still had Checks for PHP 8.1, these are not needed anymore as Laravel 10 required PHP 8.1
  • In some tests include was used, but the majority of tests already uses include_once, so I changed them to include_once

@taylorotwell
taylorotwell merged commit 13de6a0 into laravel:10.x Sep 15, 2023
@Jubeki
Jubeki deleted the remove-php81-check branch September 15, 2023 13:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@driesvints driesvints driesvints approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Jubeki @driesvints @taylorotwell