Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Issue #3277704: Remove PHPUNIT_75 constant in Rector #197

Merged
merged 1 commit into from
May 24, 2022

Conversation

FlorentTorregrosa
Copy link

Description

Remove usage of non-existing constant in rector 0.12.22

Drupal.org issue

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

@@ -11,7 +11,6 @@

$containerConfigurator->import(PHPUnitSetList::PHPUNIT_60);
$containerConfigurator->import(PHPUnitSetList::PHPUNIT_70);
$containerConfigurator->import(PHPUnitSetList::PHPUNIT_75);
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is the replacement? We need this as it fixes PHPUnit code for folks.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Answering for myself:

There is no replacement value. Removed in rector/rector-phpunit:0.11.26 due to it being a complex upgrade path.

@mglaman
Copy link
Collaborator

mglaman commented May 3, 2022

We can work on this once #195 lands and is green. This was removed in a patch release without deprecation.

@mglaman
Copy link
Collaborator

mglaman commented May 18, 2022

If this was rebased against main, it should pass. I cannot do that, however.

@FlorentTorregrosa
Copy link
Author

Hi,

Thanks @mglaman for your review.

PR rebased

@agentrickard agentrickard merged commit 5180fad into palantirnet:main May 24, 2022
@FlorentTorregrosa FlorentTorregrosa deleted the 3277704 branch May 24, 2022 15:19
@FlorentTorregrosa
Copy link
Author

Thanks for the merge :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants