- Issue created by @mparker17
- Merge request !28Issue #3465600 by mparker17, dcam, jackfoust, jenna.tollerson: Drop support for D9, D10.0, D10.1 β (Merged) created by mparker17
- π¨π¦Canada mparker17 UTC-4
Putting into "Needs review".
If you use sitemap on an older version of PHP or Drupal (especially any version of Drupal 9, Drupal 10.0. Drupal 10.1, and/or Drupal 10.2), we want to hear from you in this issue!
- πΊπΈUnited States dcam
I don't use Sitemap on versions that are EoL, but I am a fellow contrib module maintainer. For a long time I left compatibility for old versions of Drupal Core in my modules, even back to D8. It's only been in the last month while prepping for the D11 release that I finally stopped that practice.
My biggest concern at the time was whether I should drop support and then issue a major release or if it could be done in a minor release. I've seen both approaches in other modules. My strict reading of semver suggested that it would necessitate a new major release and that's why I'd always held off. So I did a search for info and found π± Guidelines for semantic versioning and Drupal core support Needs review . I found myself agreeing with the side that says "drop support in a minor release." Now my thinking is that declaring your compatibility with Core isn't the same as declaring it as a dependency. So it's ok to say "We're no longer compatible with these versions" in a minor release. And as you said, we can't test against older versions, so you can't make a guarantee about compatibility anyway. With that in mind it could even be harmful to end users if you continue to declare that compatibility.
The end result for me was that I dropped EoL Core support for all my modules. I don't know if that perspective will assist you in making your decision, but I hope that I helped a little.
- πΊπΈUnited States jackfoust
It would be great to see 10.2.x supported until its security coverage expires in December 2024, but I understand the need to move forward!
- πΊπΈUnited States jenna.tollerson Atlanta, Georgia, USA
It would be great to see 10.2.x supported until its security coverage expires in December 2024, but I understand the need to move forward!
Ditto to this. We try to stay relatively up-to-date, but not everyone can move to the latest and greatest at the same pace. :)
- π¨π¦Canada mparker17 UTC-4
Thank you for the feedback everyone!
I have no issues continuing to support 10.2.x until its security support ends (I honestly didn't know it was still supported!).
I'll update the issue title, issue summary, and merge request accordingly.
I'm going to leave this issue open for a bit longer to gather more responses.
- Status changed to Needs review
4 months ago 1:25pm 13 August 2024 - π¨π¦Canada mparker17 UTC-4
I discovered today that our tests on D9 no longer work because
\Drupal\Tests\field\Traits\EntityReferenceFieldCreationTrait
doesn't exist in that branch, so we don't really know if we break support for D9. I've added this to the issue summary.Also moving to "Needs review" to encourage the community to review it!
- Status changed to RTBC
4 months ago 2:30pm 13 August 2024 - πΊπΈUnited States dcam
I discovered today that our tests on D9 no longer work because \Drupal\Tests\field\Traits\EntityReferenceFieldCreationTrait doesn't exist in that branch...
If part of the code can't be run on D9, then the new version is officially incompatible with D9. A top module like this one likely has orgs running the automated tests as part of their deployment process. If any of them happen to be on D9, then those tests will fail. It's time to remove compatibility.
I reviewed the MR and it looks good to me. This is RTBC.
- π¨π¦Canada mparker17 UTC-4
Thanks for the feedback everyone! I'll merge the code shortly.
-
mparker17 β
committed af1e5396 on 8.x-2.x
Issue #3465600 by mparker17, dcam, jackfoust, jenna.tollerson: Drop...
-
mparker17 β
committed af1e5396 on 8.x-2.x
- Status changed to Fixed
4 months ago 10:19pm 13 August 2024 Automatically closed - issue fixed for 2 weeks with no activity.