Skip to content

update php to 8.2#35

Merged
ondrajodas merged 5 commits intomasterfrom
CM-779-ondra
Dec 8, 2023
Merged

update php to 8.2#35
ondrajodas merged 5 commits intomasterfrom
CM-779-ondra

Conversation

@ondrajodas
Copy link
Contributor

@ondrajodas ondrajodas marked this pull request as ready for review December 8, 2023 09:06
composer.json Outdated
"extra": {
"patches": {
"keboola/csv": [
"patches/keboola-csv-src-csvreader-php.patch"
Copy link
Contributor

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?

@ondrajodas
Copy link
Contributor Author

vidíš to... já tenhle PR měl připravený tak jsem tohle už nekontroloval... jdu na to

@ondrajodas
Copy link
Contributor Author

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"
Copy link
Contributor

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
Copy link
Contributor

Choose a reason for hiding this comment

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

a tohle?

@ondrajodas
Copy link
Contributor Author

ty vole však jsem to odinstaloval :D jaktože to tam pořád je :D

Copy link
Contributor

@AdamVyborny AdamVyborny left a comment

Choose a reason for hiding this comment

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

👍

@ondrajodas ondrajodas merged commit 0902500 into master Dec 8, 2023
@ondrajodas ondrajodas deleted the CM-779-ondra branch December 8, 2023 10:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants