8000 [Routing] Add tests for loading PSR-4 classes from PHP files by derrabus · Pull Request #48010 · symfony/symfony · GitHub
[go: up one dir, main page]
Skip to content

[Routing] Add tests for loading PSR-4 classes from PHP files #48010

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Oct 27, 2022
Merged

[Routing] Add tests for loading PSR-4 classes from PHP files #48010

merged 1 commit into from
Oct 27, 2022

Conversation

derrabus
Copy link
Member
Q A
Branch? 6.2
Bug fix? no
New feature? no
Deprecations? no
Tickets N/A
License MIT
Doc PR N/A

Follow-up to #47916, #47943

This PR adds more tests, demonstrating how to trigger the new PSR-4 loader from a PHP config file.

@carsonbot carsonbot changed the title Add tests for loading PSR-4 classes from PHP files [Routing] Add tests for loading PSR-4 classes from PHP files Oct 26, 2022
@carsonbot carsonbot added this to the 6.2 milestone Oct 26, 2022
@derrabus derrabus mentioned this pull request Oct 26, 2022
Merged
@fabpot
Copy link
Member
fabpot commented Oct 27, 2022

Thank you @derrabus.

@fabpot fabpot merged commit b87d6f9 into symfony:6.2 Oct 27, 2022
@derrabus derrabus deleted the tests/php-psr4-route-loading branch October 27, 2022 06:53
javiereguiluz added a commit to javiereguiluz/symfony-docs that referenced this pull request Oct 27, 2022
@javiereguiluz
minor symfony#17400 [Routing] Fix syntax for loading PSR-4 routes fro…
e49b0db 
…m PHP files (derrabus)

This PR was merged into the 6.2 branch.

Discussion
----------

[Routing] Fix syntax for loading PSR-4 routes from PHP files

see symfony/symfony#48010

Commits
-------

b3709ad Fix syntax for loading PSR-4 routes from PHP files
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fabpot fabpot fabpot approved these changes

@maxbeckers maxbeckers maxbeckers approved these changes

@javiereguiluz javiereguiluz Awaiting requested review from javiereguiluz

Assignees
No one assigned
Labels
Routing Status: Reviewed
Projects
None yet
Milestone
6.2
Development

Successfully merging this pull request may close these issues.
4 participants
@derrabus @fabpot @maxbeckers @carsonbot
0