Skip to content

[#635] Added support for env variables for Drupal root. #873

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

AlexSkrypnyk
Copy link

@AlexSkrypnyk AlexSkrypnyk commented May 3, 2025

#635

Update of #702

This helps in cases when PHPStan is called outside of the main website location.

This is still relevant in cases when tooling is outside of Drupal install.

Feel free to close, but it may help someone as well.

@mglaman
Copy link
Owner

mglaman commented May 20, 2025

I'm curious what is the use case? There is no require-dev in the root repo? Or for allowing dev dependencies not blocked by Drupal's own requirement?

@AlexSkrypnyk
Copy link
Author

2 use cases:

  1. Older versions of Drupal core have limitations on the versions of the tooling they allow, so if someone wants to use more modern versions of tools - it is not possible to do so.

  2. The website is located in a different directory than the tooling like PHPStan. This is for the cases when, for example, there are multiple websites setup for testing in sub-directories and there is a separate location for all the tooling that just runs over these different setups from a central place.

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.

2 participants