Drush12 compatibility

Attaching patch

Thanks, I can confirm the patch lets me run my behat tests with this module again after updating to Drush 12!

The patch looks good.

Hmm, actually after the patch I'm still having some trouble. When I run the behat tests that issue a drush command with this module, for example

drush datetime-testing:freeze

then I get the following error:

ddev drush datetime-testing:freeze

There are no commands defined in the "datetime-testing" namespace.

I'm still investigating the cause.

Ok, I opened an MR with hernani's patch and an additional change, the datetime testing service needs to be public because otherwise drush 12 can't access it.

Made some additional changes to match the drush 12 recommendations (the sample code generated when running drush:command-file).

Added a functional test and reverted some parts to guarantee support with drush11.

The rest looks good to me.

On Drupal 10.1 with PHP 8.2, with all the MR changes after mine in #8, I keep getting an error whenever I run any datetime-testing command with drush:

There are no commands defined in the "datetime-testing" namespace.

So I'm uploading a patch here of just my changes in #8 until I can figure out which of the later changes are breaking things for me.

Uploading patch with small changes for local testing.

@ptmkenny does it keep happening with last patch?

Sorry to be late getting back about this. Yes, it is working for me now with drush 12, so I'm marking this fixed since it's been committed.

Automatically closed - issue fixed for 2 weeks with no activity.

ptmkenny → closed merge request !8