Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Usage of symfony-cmf/routing dependency deprecated #214

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

mglaman
Copy link
Collaborator

@mglaman mglaman commented Aug 24, 2022

Description

Replaces calls to symfony-cmf/routing interfaces or classes per https://www.drupal.org/node/3151009

To Test

  • Add steps to test this feature

Drupal.org issue

https://www.drupal.org/project/rector/issues/3294535

@mglaman
Copy link
Collaborator Author

mglaman commented Aug 24, 2022

 ------ ------------------------------------------------------------------------- 
  Line   src/Rector/ConstFetch/SymfonyCmfRouteObjectInterfaceConstantsRector.php  
 ------ ------------------------------------------------------------------------- 
  61     Class Symfony\Cmf\Component\Routing\RouteObjectInterface not found.      
         💡 Learn more at https://phpstan.org/user-guide/discovering-symbols      
  66     Class Drupal\Core\Routing\RouteObjectInterface not found.                
         💡 Learn more at https://phpstan.org/user-guide/discovering-symbols      
 ------ ------------------------------------------------------------------------- 

Needs stubs for these files for PHPStan.

@agentrickard
Copy link
Contributor

@mglaman More odd conflicts

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants