Fabian Bircher

Results 11 comments of Fabian Bircher

I support this patch :+1:

I don't know if I should open a new PR, but I found another php 8.1 deprecation warning, fixed similar to this PR, this is the change: ``` diff --git...

I just synced the containers to a new browser and the original order is totally messed up and there is no way to manually re-order them. Maybe there is a...

The magic was added to drush here: https://github.com/drush-ops/drush/pull/4061 when exporting simply use the export storage and when importing use the import transformer to transform the storage before passing it on...

Hi, thanks for the PR, but I think this is the wrong approach. Update hooks (and hook update N) should definitely be allowed to change configuration. The workflow should be:...

I had a small chat with @weitzman and I think it would be helpful to illustrate why I think failing on a perfectly ok scenario is not the right solution....

The pull request does not what you describe here because you say that `2. Runs update hooks and config import as we always have.`. But in the Pull request the...

But I am arguing that this is still wrong! Because I might have removed 🟢 on purpose from the "tracked config" during site building after I ran the update hooks...

+1 Actually the place where the `$@` is is wrong since it will have the first argument duplicated, rather it should be `FWD="$@"` on line 12

I think it would probably be possible. But I am not sure that it would not create more problems than it is worth. If you call this new static get...