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

Add support for Laravel 11 #1117

Merged
merged 8 commits into from
Feb 18, 2024
Merged

Add support for Laravel 11 #1117

merged 8 commits into from
Feb 18, 2024

Conversation

mfn
Copy link
Collaborator

@mfn mfn commented Feb 18, 2024

Summary

  • Also add support for phpunit 10
    • this required fixing a test and adapting the suffix test pattern
  • remove phpstan from tests (version conflicts)
  • cancel GHA workflows if a new push arrives

For the integration tests, I needed to use 11.x-dev for now


Type of change:

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update
  • Misc. change (internal, infrastructure, maintenance, etc.)

Checklist:

  • Existing tests have been adapted and/or new tests have been added
  • Add a CHANGELOG.md entry
  • Update the README.md
  • Code style has been fixed via composer fix-style

@mfn mfn self-assigned this Feb 18, 2024
mfn added 7 commits February 18, 2024 12:35
This avoid the hundreds of warnings about classes phpunit can't find,
because they're in fact not test classes.

E.g.:
```
There were 116 PHPUnit test runner warnings:

1) Class ExampleNestedValidationInputObject cannot be found in /Users/neo/src/swatio/tmp/mfn/graphql-laravel/tests/Unit/AliasArguments/Stubs/ExampleNestedValidationInputObject.php

2) Class ExampleType cannot be found in /Users/neo/src/swatio/tmp/mfn/graphql-laravel/tests/Unit/AliasArguments/Stubs/ExampleType.php
…
```
To avoid composer version dependency conflicts
@@ -50,8 +50,8 @@
"mockery/mockery": "^1.2",
"phpstan/phpstan": "1.10.40",
"larastan/larastan": "2.6.4",
"orchestra/testbench": "^7.0|^8.0",
"phpunit/phpunit": "^9",
"orchestra/testbench": "^7.0|^8.0|^9.0|^9.x-dev",
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

^9.0 isn't available yet, hence we also add ^9.x-dev, ought to be removed once 9 is out

@mfn mfn merged commit b181329 into rebing:master Feb 18, 2024
@mfn mfn deleted the mfn-laravel-11 branch February 18, 2024 11:39
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.

1 participant