search

jhedstrom / DrupalDriver

A collection of drivers for controlling Drupal.
GNU General Public License v2.0
64 stars 95 forks source link

Dev dependencies #223

Closed Berdir closed 4 years ago

Berdir commented 4 years ago

Updating necessary dev dependencies to try and get it working on more recent php versions.

only have php 7.3 locally, might need to allow 8 too which might require more changes.

Berdir commented 4 years ago

the drupal/coder update might not be necessary, but I think the coding standard improvements make sense, although likely to conflict with some other PR's.

Berdir commented 4 years ago

Ok, this passes despite a deprecation warning in jakub-onderka/php-parallel-lint which doesn't seem to be really maintained anymore. Might be possible to drop this, speed doesn't really seem like an issue here?

jhedstrom commented 4 years ago

Might be possible to drop this, speed doesn't really seem like an issue here? Sure, we can do that in a follow-up, but I merged this for now.