Skip to content

[PhpUnitBridge] ExcludeList usage for PHPUnit 9.4 #37513

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

Conversation

gennadigennadigennadi
Copy link
Contributor

Q A
Branch? 4.4
Bug fix? yes
New feature? no
Deprecations? no
Tickets #37344
License MIT

Added: ExcludeList usage for PHPUnit >= 9.3

@nicolas-grekas
Copy link
Member

Great thanks. Can you please target branch 3.4?

@nicolas-grekas nicolas-grekas added this to the 3.4 milestone Jul 7, 2020
@gennadigennadigennadi
Copy link
Contributor Author

gennadigennadigennadi commented Jul 7, 2020

The fix (CommandForV9) for PHPUnit 9 is missing inside the 3.4 Branch, I have not checked if 3.4 is capable of running PHPUnit 9.

Are the fixes for PHPUnit 9 not required in Symfony 3.4? At least I have not not seen them inside the 3.4 Branch.

@nicolas-grekas nicolas-grekas modified the milestones: 3.4, 4.4 Jul 8, 2020
@nicolas-grekas
Copy link
Member

The fix (CommandForV9) for PHPUnit 9 is missing inside the 3.4 Branch

Oh, OK for 4.4 then!

@@ -254,7 +254,11 @@
if (!class_exists('SymfonyBlacklistPhpunit', false)) {
class SymfonyBlacklistPhpunit {}
Copy link
Member

@nicolas-grekas nicolas-grekas Jul 8, 2020

Choose a reason for hiding this comment

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

can you rename this class also please?
SymfonyExcludeListPhpunit

Copy link
Member

Choose a reason for hiding this comment

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

(fixed)

@nicolas-grekas nicolas-grekas changed the title Add: ExcludeList usage for PHPUnit 9.4 [PhpUnitBridge] ExcludeList usage for PHPUnit 9.4 Jul 8, 2020
@nicolas-grekas nicolas-grekas force-pushed the fix/use-excludelist-instead-of-blacklist branch from aa2088f to f59915d Compare July 8, 2020 07:59
@nicolas-grekas
Copy link
Member

Thank you @gennadigennadigennadi.

@nicolas-grekas nicolas-grekas merged commit c702af4 into symfony:4.4 Jul 8, 2020
@gennadigennadigennadi gennadigennadigennadi deleted the fix/use-excludelist-instead-of-blacklist branch July 8, 2020 08:29
This was referenced Jul 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants