Skip to content
This repository has been archived by the owner on Jan 8, 2021. It is now read-only.

Add php8 as CI target #77

Closed

Conversation

ChristophWurst
Copy link
Contributor

@ChristophWurst ChristophWurst commented Dec 30, 2020

We might have to also adjust the distro as precise is a bit outdated but let's see

Once I know the tests indeed fail I'll tackle #74

@ChristophWurst
Copy link
Contributor Author

FYI Travis is kind of shutting down. I suggest moving to Github Actions. If you need help please let me know ✌️

Copy link
Contributor

@nicolas-grekas nicolas-grekas left a comment

Choose a reason for hiding this comment

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

FYI Travis is kind of shutting down. I suggest moving to Github Actions. If you need help please let me know v

I hopefully migrated to travis-ci.com, lets see how it goes.

.travis.yml Show resolved Hide resolved
composer.json Outdated Show resolved Hide resolved
@ChristophWurst
Copy link
Contributor Author

I hopefully migrated to travis-ci.com, lets see how it goes.

don't get your hopes up. they limit the "open source credit" that you get. and while the docs say you can get credit for open source, they actually shut that program down. so either you pay or you have to use something else. it's affecting us a lot at Nextcloud and they did that without any warning. just as a heads-up.

Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
@ChristophWurst
Copy link
Contributor Author

Ok, so this runs but it doesn't pass. but I see other builds did just fail as much: https://travis-ci.org/github/tchwork/utf8/builds/752168267. @nicolas-grekas do you have any idea for how to support such a wide range of php versions with phpunit?

Some new failures include

1) Patchwork\Tests\PHP\Shim\IntlTest::testGrapheme_substr

Failed asserting that '' is identical to false.

/home/travis/build/tchwork/utf8/tests/PHP/Shim/IntlTest.php:89

2) Patchwork\Tests\PHP\Shim\IntlTest::testGrapheme_strpos

Failed asserting that 0 is identical to false.

/home/travis/build/tchwork/utf8/tests/PHP/Shim/IntlTest.php:122

3) Patchwork\Tests\PHP\Shim\MbstringTest::testmb_stubs

mb_encode_mimeheader() is bugged. Please use iconv_mime_encode() instead

Failed asserting that true is false.

/home/travis/build/tchwork/utf8/tests/PHP/Shim/MbstringTest.php:31

4) Patchwork\Tests\PHP\Shim\MbstringTest::testmb_strpos

Failed asserting that 0 is identical to false.

/home/travis/build/tchwork/utf8/tests/PHP/Shim/MbstringTest.php:113

5) Patchwork\Tests\PHP\Shim\MbstringTest::testmb_strpos_empty_delimiter

The previous line should trigger a warning (Empty delimiter)

Failed asserting that true is false.

/home/travis/build/tchwork/utf8/tests/PHP/Shim/MbstringTest.php:148

@nicolas-grekas
Copy link
Contributor

Honest question: are you really using this package? Why not https://github.com/symfony/polyfill-mbstring/ and the likes instead?

@ChristophWurst
Copy link
Contributor Author

Well this is embarrassing for me. I checked in detail and while we still depend on this package, another package recently also added the intl-normalizer polyfill from Symfony into our project and so if I disable the intl extension I actually get a conflict because this package tries to re-declare a already polyfilled function.

Hence we can close this as we indeed won't use the package anymore once my change removes it from our project.

Thanks, in any case and cheers from Nextcloud ✌️

@ChristophWurst ChristophWurst deleted the enhancement/php8-ci branch January 7, 2021 15:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants