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.

Sign up for GitHub

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

Jump to bottom

add php8 compatibility without BC (supports php5, 7, 8, 8.3, 8.4) #382

Open

HenryVolkmer wants to merge 4 commits into kartik-v:master from HenryVolkmer:bugfix/php8.4-compatibility

HenryVolkmer commented Jan 3, 2025

Scope

This pull request fixes #378 and ensures php8 compatibility.

Changes

(bug Declaration of kartik\export\ExportWriterPdf::createExternalWriterInstance($config = []) not compatible with PhpSpreadsheet v2 #378): satisfy input/return typehints
ensure php8 compatibility

Related Issues

Henry Volkmer added 3 commits

January 3, 2025 15:57


          - Bugfix kartik-v#378

eb00152


          - php>=8 compatibility

748a169


          - add CHANGE.md

Author

HenryVolkmer commented Jan 3, 2025

no BC, module will run on php5 til php8.4

HenryVolkmer changed the title ~~add php8 compatibility~~ add php8 compatibility without BC (supports php5, 7, 8, 8.3, 8.4)


          remove obsolete debug code

311032b

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet