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

PsrAutoloadingFixer - Fix PSR autoloading outside configured directory #5421

Merged
merged 2 commits into from
Mar 20, 2021

Conversation

kelunik
Copy link
Contributor

@kelunik kelunik commented Jan 9, 2021

kelunik added a commit to amphp/php-cs-fixer-config that referenced this pull request Jan 11, 2021
@keradus keradus changed the base branch from 2.17 to 2.18 January 24, 2021 21:33
@SpacePossum
Copy link
Contributor

Thanks for the PR as bug fix 👍

Maybe we should validate that the configured directory exists or at least check the result of the realpath call?
I think we can also call the realpath just one time for each run (so not for each file that is processed).

@SpacePossum SpacePossum added this to the 2.18.3 milestone Mar 10, 2021
@keradus keradus modified the milestones: 2.18.3, 2.18.4 Mar 10, 2021
@clxmstaab
Copy link
Contributor

should/will this PR als fix #5537 ?

@keradus
Copy link
Member

keradus commented Mar 12, 2021

@clxmstaab no,
reason in #5537 (comment)

@keradus keradus force-pushed the fix-psr-autoloading branch from c20aafb to 02d1ade Compare March 19, 2021 13:28
@coveralls
Copy link

coveralls commented Mar 19, 2021

Coverage Status

Coverage increased (+0.003%) to 91.849% when pulling 5d2d99d on kelunik:fix-psr-autoloading into d3aaf70 on FriendsOfPHP:2.18.

@keradus keradus force-pushed the fix-psr-autoloading branch from 02d1ade to 55a9991 Compare March 19, 2021 13:51
@keradus
Copy link
Member

keradus commented Mar 19, 2021

I let myself to apply suggestion of @SpacePossum

@keradus keradus changed the title Fix psr autoloading outside configured directory PsrAutoloadingFixer - Fix PSR autoloading outside configured directory Mar 20, 2021
@keradus
Copy link
Member

keradus commented Mar 20, 2021

Thank you @kelunik.

@keradus keradus merged commit 06f764e into PHP-CS-Fixer:2.18 Mar 20, 2021
@kelunik
Copy link
Contributor Author

kelunik commented Mar 20, 2021

Thank you, we can finally enable the rule again. :-)

@kelunik kelunik deleted the fix-psr-autoloading branch April 3, 2021 09:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unwanted renaming
6 participants