Skip to content

use composer_autoload_path if possible#1478

Merged
gennadigennadigennadi merged 3 commits into
4.0.xfrom
use-composer-autoload-global
Jul 17, 2025
Merged

use composer_autoload_path if possible#1478
gennadigennadigennadi merged 3 commits into
4.0.xfrom
use-composer-autoload-global

Conversation

@gennadigennadigennadi

@gennadigennadigennadi gennadigennadigennadi commented Jul 15, 2025

Copy link
Copy Markdown
Collaborator

fixes: #1475

@gennadigennadigennadi gennadigennadigennadi force-pushed the use-composer-autoload-global branch from 0da8a37 to f7cb845 Compare July 15, 2025 20:09
@gennadigennadigennadi

Copy link
Copy Markdown
Collaborator Author

Should we use instead of getcwd() something like phpunit: https://github.com/sebastianbergmann/phpunit/blob/a62124f38b2e3c17b2e338dfe5ff9d4ff6279363/phpunit#L50.

Right now we don't early return, could this cause bugs? E.g.: registering multiple autoload.php

@gennadigennadigennadi gennadigennadigennadi force-pushed the use-composer-autoload-global branch from 758fb68 to ce0a5e9 Compare July 17, 2025 21:53
add phpunit style autoload.php resolution]
@gennadigennadigennadi gennadigennadigennadi force-pushed the use-composer-autoload-global branch from b3bafae to f6ffcd5 Compare July 17, 2025 22:15
@gennadigennadigennadi gennadigennadigennadi marked this pull request as ready for review July 17, 2025 22:32
@gennadigennadigennadi gennadigennadigennadi merged commit 089268e into 4.0.x Jul 17, 2025
18 checks passed
@gennadigennadigennadi gennadigennadigennadi deleted the use-composer-autoload-global branch July 17, 2025 22:35

@patrickkusebauch patrickkusebauch left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

👍 LGTM

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.

Fix vendor/autoload.php

2 participants