[Lunr.Config] General: PHP 8.x compatibility

Review Request #821 — Created Feb. 15, 2022 and submitted June 15, 2022, 4:32 p.m.

Description

General: PHP 8.x compatibility

Testing Done

Issues

Description	From	Last Updated
Composer changes conflict with https://reviews.lunr.nl/r/754/	pprkut	June 14, 2022, 8:42 a.m.
No?	pprkut	March 28, 2022, 2:57 p.m.
I still don't get why this is needed. There's no difference between not specifying a return type and : mixed...	pprkut	May 2, 2022, 9:36 a.m.

composer.json (Diff revision 1)

Show all issues

No?

If we do this it will be AFAIK, since it has types in 8.0 but not in 7.x

Well then we can't. Removing support for PHP 7 isn't in the cards right now.

Revision 2 (+145 -134)

	composer.lock
	composer.json
	src/Lunr/Core/Configuration.php

Show all issues

Composer changes conflict with https://reviews.lunr.nl/r/754/

I'll rebase the remaining change whenever one is merged

src/Lunr/Core/Configuration.php (Diff revision 2)

Show all issues

I still don't get why this is needed. There's no difference between not specifying a return type and : mixed...

Because in 8.x the base class does have the return type, so omitting it means it's incompatible with the interface.

Ship it!

You have a pending review.