Problem/Motivation

Followup to #3197729: Remove forward/bc layer for Symfony Definition::setDeprecated() method

We missed a BC shim in DeprecatedServicePass, we can remove it now.

Steps to reproduce

Proposed resolution

Remove the code that supports Symfony 4.

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

CommentFileSizeAuthor
#2 3265120-2.patch1.82 KBlongwave
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

longwave created an issue. See original summary.

longwave’s picture

longwave
he/him
Primary language English
Location UK
CreditAttribution: longwave at Full Fat Things commented
Status: Active » Needs review
FileSize
3265120-2.patch1.82 KB
daffie’s picture

daffie CreditAttribution: daffie commented
Status: Needs review » Reviewed & tested by the community

Looks good to me.

  • catch committed a209347 on 10.0.x 
    Issue #3265120 by longwave: Remove Symfony 4 BC shim from...
catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch at Third and Grove commented
Status: Reviewed & tested by the community » Fixed

Committed a209347 and pushed to 10.0.x. Thanks!

Status: Fixed » Closed (fixed)

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