Moshe Weitzman

Results 474 comments of Moshe Weitzman

Can we link to Drupal's docs instead. Its not really Drush's place to explain this. deploy/updatedb is just a runner.

Lets reopen if a link to drupal docs arises.

I'm neutral on this. A coder has to lookup the service name and type hint - thats not nothing. This question would benefit a lot from a [MultiSearch Prompt](https://laravel.com/docs/11.x/prompts#multisearch). Please...

It is the responsibility of the field type (i.e. text) to generate a sample value and to respect the max/min and such. See https://github.com/drush-ops/drush/blob/68e5d4b4ced0846b2f74064e5399ad31b7a27225/src/Commands/sql/sanitize/SanitizeUserFieldsCommands.php#L74. Maybe there is a bug here...

[3 PHPStan errors are persisting](https://app.circleci.com/pipelines/github/drush-ops/drush/7242/workflows/d25de119-2c7a-43bf-b644-ae8fec191a65/jobs/43204/tests). They are about loadInclude(). Help wanted. You can still push to the source branch @andyg5000. P.S. Sorry for the awkward branch naming - my `gh...

Those 2 fixes are already in the PR. I am seeing the same failures locally. See below. And I dont see caching that is setup for this job in CI....

Maybe retry with the surce branch of this new PR - andyg5000/13.x (sorry for the confusing name). Otherwise I dont know why this passes for you and fails for me...

The last results I pasted were with --debug. Each filename was output before the report that I pasted above.

I deleted my DDEV build and started from scratch - same errors. I would be very surprised if this is a caching issue.

I ran with --xdebug and I saw the includes getting require_once() successfully but still those symbols are not found. Its a mystery.