Skip to content

Conversation

@greg0ire
Copy link
Member

@greg0ire greg0ire commented Jul 30, 2020

This should help getting the PHP 8 build closer to a successful one. There is now a new error in the PHP 8 build, but the deprecations are gone.

Fixes #340

@greg0ire greg0ire marked this pull request as draft July 30, 2020 19:33
@greg0ire

This comment has been minimized.

greg0ire added 2 commits July 30, 2020 22:24
This should help getting the PHP 8 build closer to a successful one.
Fixes doctrine#340
That is a version where we can installed newer versions of PHPUnit, and
use methods that may not exist otherwise.
@greg0ire greg0ire marked this pull request as ready for review July 30, 2020 20:32
@stof
Copy link
Member

stof commented Jul 31, 2020

is the fatal error on PHP 8 coming from an issue in our code/testsuite or from PHPUnit ?

@greg0ire
Copy link
Member Author

greg0ire commented Jul 31, 2020

is the fatal error on PHP 8 coming from an issue in our code/testsuite or from PHPUnit ?

phpunit evals code that seems to come from us (I think, it might be very indirect): https://github.com/sebastianbergmann/phpunit/blob/9e202b52bc63ddca3d86c374b2f6c726561d9cbb/src/Framework/MockObject/MockClass.php#L45

But I can't find occurences of mixed|null in our code.

@greg0ire
Copy link
Member Author

Actually it might not come from our code:

$class = $this->getMockBuilder(ReflectionClass::class)
->disableOriginalConstructor()
->getMock();

@greg0ire
Copy link
Member Author

Investigating that separately at #343

@greg0ire greg0ire mentioned this pull request Jul 31, 2020
@greg0ire greg0ire merged commit bfa814d into doctrine:1.10.x Jul 31, 2020
@greg0ire greg0ire deleted the upgrade-phpunit branch July 31, 2020 12:49
@greg0ire greg0ire added this to the 1.10.4 milestone Jul 31, 2020
jkufner added a commit to smalldb/annotations that referenced this pull request Aug 11, 2020
Release [1.10.4](https://github.com/doctrine/annotations/milestone/25)

1.10.4
======

- Total issues resolved: **0**
- Total pull requests resolved: **8**
- Total contributors: **5**

 - [347: Add support for the new PHP 8 tokens for use statements](doctrine#347) thanks to @stof
 - [345: Remove calls to TestCase::at()](doctrine#345) thanks to @greg0ire
 - [343: Allow using PHPUnit 9.3](doctrine#343) thanks to @greg0ire

Improvement
-----------

 - [342: Upgrade phpunit](doctrine#342) thanks to @greg0ire
 - [332: DocParser: Improve private typehints](doctrine#332) thanks to @jkufner

bug
---

 - [341: Make type in phpdoc resolvable](doctrine#341) thanks to @greg0ire

Documentation
-------------

 - [338: update annotation IDE annotation with current links](doctrine#338) thanks to @Haehnchen

Documentation,Improvement
-------------------------

 - [337: Replace "blacklist" terminology with "ignore"](doctrine#337) thanks to @albe
Yurunsoft added a commit to Yurunsoft/doctrine-annotations that referenced this pull request Nov 29, 2020
Release [1.10.4](https://github.com/doctrine/annotations/milestone/25)

1.10.4
======

- Total issues resolved: **0**
- Total pull requests resolved: **8**
- Total contributors: **5**

 - [347: Add support for the new PHP 8 tokens for use statements](doctrine#347) thanks to @stof
 - [345: Remove calls to TestCase::at()](doctrine#345) thanks to @greg0ire
 - [343: Allow using PHPUnit 9.3](doctrine#343) thanks to @greg0ire

Improvement
-----------

 - [342: Upgrade phpunit](doctrine#342) thanks to @greg0ire
 - [332: DocParser: Improve private typehints](doctrine#332) thanks to @jkufner

bug
---

 - [341: Make type in phpdoc resolvable](doctrine#341) thanks to @greg0ire

Documentation
-------------

 - [338: update annotation IDE annotation with current links](doctrine#338) thanks to @Haehnchen

Documentation,Improvement
-------------------------

 - [337: Replace "blacklist" terminology with "ignore"](doctrine#337) thanks to @albe

# gpg: Signature made Tue Aug 11 20:55:32 2020
# gpg:                using DSA key 1BEDEE0A820BC30D858F9F0C2C3A645671828132
# gpg: Can't check signature: No public key

# Conflicts:
#	composer.json
#	lib/Doctrine/Common/Annotations/Annotation/Attributes.php
#	lib/Doctrine/Common/Annotations/DocParser.php
#	tests/Doctrine/Tests/Common/Annotations/AbstractReaderTest.php
#	tests/Doctrine/Tests/Common/Annotations/AnnotationReaderTest.php
#	tests/Doctrine/Tests/Common/Annotations/DocParserTest.php
#	tests/Doctrine/Tests/Common/Annotations/FileCacheReaderTest.php
#	tests/Doctrine/Tests/Common/Annotations/SimpleAnnotationReaderTest.php
#	tests/Doctrine/Tests/Common/Annotations/Ticket/DCOM55Test.php
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.

PHPUnit needs to be upgrade

3 participants