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

Add Drush command for configuring PHPStan #391

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

Add Drush command for configuring PHPStan #391

wants to merge 6 commits into from

Conversation

mglaman
Copy link
Owner

@mglaman mglaman commented Apr 13, 2022

Fixes #343

@mglaman
Copy link
Owner Author

mglaman commented Apr 13, 2022

Instead of being used to dump phpstan.neon, could be a supplemental file that gets included into the phpstan.neon for drupal related items.

@ondrejmirtes
Copy link
Contributor

You don’t need to use nette/neon to write the file, you can output it like a human would, by echoing tabs and newlines 😊

@mglaman
Copy link
Owner Author

mglaman commented Apr 14, 2022

You don’t need to use nette/neon to write the file, you can output it like a human would, by echoing tabs and newlines 😊

😅 Yeah, thinking back it's not that bad. Especially since it's just "is string or array?" But apparently it was too much for my end of day brain

@mglaman
Copy link
Owner Author

mglaman commented May 4, 2022

Fails due to Symfony v4 and v6 compat, see https://github.com/drush-ops/drush/pull/5108/files#diff-fcb54a74a331c6112bd5cf5e32f08a4d0a39f42c16833b899b4fac0339d453c7

Need to call \Drush\Preflight\Preflight::loadSymfonyCompatabilityAutoloader – Drush 11 only!

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.

Provide a Drush command to generate configuration
2 participants