-
Notifications
You must be signed in to change notification settings - Fork 0
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
update php to 8.2 #35
Conversation
composer.json
Outdated
"extra": { | ||
"patches": { | ||
"keboola/csv": [ | ||
"patches/keboola-csv-src-csvreader-php.patch" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Proč nemergneš tohle, když to je připravený: keboola/php-csv#55?
vidíš to... já tenhle PR měl připravený tak jsem tohle už nekontroloval... jdu na to |
je to tam |
composer.json
Outdated
"keboola/datadir-tests": "^5.2", | ||
"phpstan/phpstan": "^0.12.14" | ||
"symfony/process": "^5.4", | ||
"symplify/vendor-patches": "^11.2" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tak to už můžeš vyhodit tohle, ne?
composer.json
Outdated
"optimize-autoloader": true, | ||
"allow-plugins": { | ||
"dealerdirect/phpcodesniffer-composer-installer": true, | ||
"cweagans/composer-patches": false |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
a tohle?
ty vole však jsem to odinstaloval :D jaktože to tam pořád je :D |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
https://keboola.atlassian.net/browse/CM-779