- Issue created by @moshe weitzman
- First commit to issue fork.
- First commit to issue fork.
- 🇮🇳India chandu7929 Pune
Updated variable name as requested, hence moving in NR.
- 🇮🇳India rajeshreeputra Pune
Rebased and added to 🌱 Planned for 1.21 version release Active .
- 🇮🇳India vishalkhode
Reviewed changes, looks good to me. Hence, RTBC.
Note: Core Version requirement has been updated and Drupal Core 8 support is dropped. Probably this requires a new minor release. - 🇯🇵Japan ptmkenny
This is tagged "needs tests" and there are no tests in the MR, so it cannot be RTBC.
- 🇯🇵Japan ptmkenny
As shown in the MR for 📌 Remove support for Drupal < 9.1 Needs review , simply dropping Drupal 8 + 9 support is not sufficient here; there is also code that handles a special case for D8 that needs to be removed. I think Drupal 8/9 support should be removed in a separate issue before this is committed, and this issue should be made dependent on that one.
- First commit to issue fork.
- 🇺🇸United States japerry KVUO
Isn't there a way for modern drush commands to work alongside the old drush versions? Some sites have constraints keeping them on earlier versions of drush, so I'm not sure such a core module like key should be moving away from supporting those quite yet.