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

Adding DIR to include statement to fix issue with pathing #8564

Merged
merged 1 commit into from Mar 26, 2021

Conversation

cybercitizen7
Copy link

@cybercitizen7 cybercitizen7 commented Mar 26, 2021

Fixes #8563

@cybercitizen7
Copy link
Author

This PR should fix the reported issue here:
#8563

I do not have permission to set Linked Issues on this repository.

@cybercitizen7
Copy link
Author

@greg0ire if you will be reviewing this PR. Could you please do a proper test whether this is the best fix?

@greg0ire greg0ire linked an issue Mar 26, 2021 that may be closed by this pull request
@greg0ire greg0ire added the Bug label Mar 26, 2021
@greg0ire
Copy link
Member

We don't have functional tests in that project, so the manual test you did will have to do I'm afraid.

@greg0ire greg0ire merged commit 3f3de70 into doctrine:2.8.x Mar 26, 2021
@greg0ire
Copy link
Member

Thanks @cybercitizen7 !

@greg0ire greg0ire added this to the 2.8.3 milestone Mar 26, 2021
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.

vendor/bin/doctrine won't work, reporting missing 'doctrine.php'
2 participants