Compatibility with Migrate Tools 6.0.x-dev

Agree with RTBC. Just encountered this issue and the merge request fixed it for me.

Just encountered this issue and the merge request seems to have resolved the issue here too

After applying the attached patch (extracted safely from MR !33), the following fatal error is resolved:

PHP Fatal error: Declaration of Drupal\path_redirect_import\Drush\Commands\PathRedirectImportCommands::create(ContainerInterface $container): Drupal\migrate_tools\Drush\Commands\MigrateToolsCommands must be compatible with ...::create(ContainerInterface $container): static

To keep this from happening a third time, I suggest tighter dependency restrictions in the info and composer.json files.

And could automated tests have caught this?

Adding the actual error to help the search bots.

As mentioned in #8 🐛 Compatibility with Migrate Tools 6.0.x-dev Active , the patch should include changes in composer and info files to ensure that new module version including this change will work with migrate_tools ^6.1.0.

It's hard to define a tighter dependency because we don't have the exact information about when the parent module will introduce new changes in the parent class constructor.

Okay and from there on your end you'll then create new branch as appropriate? Or do we do this in 2.1.x?

Do not worry about branches. I'll take care of them. Thank you!

Okay updated info yml and composer json minimum versions, thanks!

Works as expected with Migrate Tools 6.1.0
Thanks! Moving the ticket to RTBC.

MR merged and released v2.1.1

https://www.drupal.org/project/path_redirect_import/releases/2.1.1 →

Thank you!