-
-
Notifications
You must be signed in to change notification settings - Fork 328
PHP-DI 7.0 #726
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
Merged
PHP-DI 7.0 #726
Changes from all commits
Commits
Show all changes
60 commits
Select commit
Hold shift + click to select a range
1c0a168
Drop support for phpdoc types with @Inject by moving to PHP 7.4+
mnapoli dde4627
Remove the obsolete `ignorePhpDocErrors` option
mnapoli 0f3b69f
Fix code formatting
mnapoli 15afdad
Improve type declarations following the PHP 7.4 requirement
mnapoli 3d9ac0c
Fix coding style
mnapoli 4cdea6c
Remove useless config files
mnapoli a74041b
Update dependency versions to fix tests
mnapoli 3585442
Improve support for PHP 8
mnapoli 15ef9d5
Inject via PHP 8 attributes
mnapoli 4aeb4ac
Merge pull request #738 from PHP-DI/php8-attributes
mnapoli b7d3d8f
Fix PHP 8 attributes 🤦
mnapoli 4b4e31f
Merge branch 'master' into v7
mnapoli 0ea9d81
Update CI for v7
mnapoli 2a8bb30
Support PHP 8 attributes at the parameter level
mnapoli f31f16a
Merge pull request #741 from PHP-DI/v7-php8-parameter-attributes
mnapoli a1be38d
Test and document using PHP 8 named arguments for config
mnapoli b6de725
Require PHP 8.0 and use stricter types/fix warnings
mnapoli 95a1a77
Fix psalm warnings
mnapoli 18b192f
Require PHP 8
mnapoli 91d2f1a
Psalm templates
mnapoli 20873db
More psalm fixes
mnapoli bc862ff
Replace phpstan with psalm
mnapoli 2017dc5
Fix code formatting
mnapoli 849a06a
Merge pull request #766 from PHP-DI/php8-only
mnapoli da0912c
Fixes after upping the requirement to PHP 8
mnapoli ad2ba80
Merge branch 'master' into v7
mnapoli d842943
Fix minimum stability
mnapoli 9df6976
Require psr/container 1.1 to strictly type parameters
mnapoli ec8fbdb
Remove now useless test
mnapoli 4291647
Run APCu tests in CI
mnapoli 902cc91
Support psr/container 2.0
mnapoli efa8ccd
Run APCu tests in CI
mnapoli 957f926
Allow to create the container with definitions
mnapoli afb1ca7
Remove obsolete ContainerBuilder::buildDevContainer()
mnapoli 99b0104
Improve the "Getting started" documentation
mnapoli 50819b5
Fix CS
mnapoli 9eaa05e
Merge pull request #769 from PHP-DI/simpler-container-creation
mnapoli 6cb9fa2
Merge master into v7
mnapoli d01fdf9
Simplify method
mnapoli aa0ea10
Various code fixes and improvements (PHPStorm warnings)
mnapoli 236f434
Upgrade php-cs-fixer
mnapoli 1117da7
Force strict types
mnapoli 891bd5e
Improve code style
mnapoli c3891b0
Drop support for `@Inject` annotations in favor of attributes
mnapoli 8d35559
Merge pull request #812 from PHP-DI/drop-annotations
mnapoli 35f16fc
Merge branch 'master' into v7
mnapoli 5458204
Merge remote-tracking branch 'origin/v7' into v7
mnapoli a5b25de
Merge branch 'master' into v7
mnapoli 6e8d9f5
Fix build
mnapoli a326519
Merge branch 'master' into v7
mnapoli 8948ffb
Prepare the blog post for the 7.0 release
mnapoli 921a1c8
Remove non-analyzed vendor directory from Psalm ignores
szepeviktor f540193
Merge pull request #837 from szepeviktor/patch-3
mnapoli be4a41a
Fix typo
jdreesen 39b2557
Fix typo
jdreesen ea93a1f
Merge pull request #839 from jdreesen/patch-1
mnapoli d3e0057
Correct type hints to prevent tooling like PHPStan from raising issues.
ldebrouwer-rmn 6011611
Update src/Definition/Helper/CreateDefinitionHelper.php
ldebrouwer 7a17403
Update src/Definition/Helper/CreateDefinitionHelper.php
ldebrouwer 6d9eab0
Merge pull request #841 from ldebrouwer/ldebrouwer/correct-type-hints
mnapoli File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
8000 Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You probably want PSR12 here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not interested in PSR-12 honestly. This is a rename of the file that exists on master.