Problem/Motivation

Drupal 8 is no longer supported and stopped receiving security coverage in Nov 2021.

We have a fresh enough release that still supports D8, so it's probably a good time to drop D8 support.

The alternative would be to introduce a 2.x branch, but I don't really think that makes sense without an actual plan.

CommentFileSizeAuthor
#13 3260637-13.patch7.08 KBjsacksick
#11 3260637-11.patch5.17 KBjsacksick
#9 3260637-9.patch4.71 KBjsacksick
#7 3260637-7.patch4.27 KBjsacksick
#4 3260637-3.patch3.12 KBjsacksick
#2 3260637-2.patch3.1 KBjsacksick
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jsacksick created an issue. See original summary.

jsacksick’s picture

jsacksick CreditAttribution: jsacksick at Centarro commented
Status: Active » Needs review
FileSize
3260637-2.patch3.1 KB
jsacksick’s picture

jsacksick CreditAttribution: jsacksick at Centarro commented
Issue summary: View changes
jsacksick’s picture

jsacksick CreditAttribution: jsacksick at Centarro commented
FileSize
3260637-3.patch3.12 KB

Let's allow installation with D10 while we're at it.

The last submitted patch, 2: 3260637-2.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

Status: Needs review » Needs work

The last submitted patch, 4: 3260637-3.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

jsacksick’s picture

jsacksick CreditAttribution: jsacksick at Centarro commented
Status: Needs work » Needs review
FileSize
3260637-7.patch4.27 KB

Status: Needs review » Needs work

The last submitted patch, 7: 3260637-7.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

jsacksick’s picture

jsacksick CreditAttribution: jsacksick at Centarro commented
Status: Needs work » Needs review
FileSize
3260637-9.patch4.71 KB

Status: Needs review » Needs work

The last submitted patch, 9: 3260637-9.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

jsacksick’s picture

jsacksick CreditAttribution: jsacksick at Centarro commented
Status: Needs work » Needs review
FileSize
3260637-11.patch5.17 KB
jsacksick’s picture

jsacksick CreditAttribution: jsacksick at Centarro commented

Yeah, so the patch from #11 was an attempt to fix the deprecation notice which can't be fixed because of the parent class which has the $modules variable declared as public.

jsacksick’s picture

jsacksick CreditAttribution: jsacksick at Centarro commented
Issue summary: View changes
FileSize
3260637-13.patch7.08 KB

  • jsacksick committed 0c03e4f on 8.x-1.x 
    Issue #3260637 by jsacksick: Drop D8 support.
jsacksick’s picture

jsacksick CreditAttribution: jsacksick at Centarro commented
Status: Needs review » Fixed

Committed!

Status: Fixed » Closed (fixed)

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