Skip to content

Extend support range for PHP#25

Open
juicylime wants to merge 4 commits into
laminas-api-tools:1.10.xfrom
juicylime:support-php-8.1
Open

Extend support range for PHP#25
juicylime wants to merge 4 commits into
laminas-api-tools:1.10.xfrom
juicylime:support-php-8.1

Conversation

@juicylime

@juicylime juicylime commented Nov 3, 2022

Copy link
Copy Markdown

Signed-off-by: Aman Ahlwat amanahlwat@gmail.com

Q A

| New Feature | yes

Description

Extending the support range for PHP to support ~8.0

Fixes #24

Signed-off-by: Aman Ahlwat <aman.ahlwat@grouphealth.ca>
Comment thread composer.json Outdated
@Ocramius Ocramius added the Enhancement New feature or request label Nov 3, 2022
@Ocramius Ocramius added this to the 1.10.0 milestone Nov 3, 2022
@Ocramius

Ocramius commented Nov 3, 2022

Copy link
Copy Markdown
Member

Hmm, I think this error is because CI didn't run in a while, and composer/composer:2.3.x broke BC here.

config.allow-plugins is missing here: a local composer install will highlight that.

@Ocramius

Ocramius commented Nov 3, 2022

Copy link
Copy Markdown
Member

🤔 ok, now that's where the mess starts: PHP 8.0 and 8.1 don't cleanly install at all?

Nothing to do with your work, heh.

I see that laminas/laminas-hydrator support could be expanded to ^4 perhaps: ^3 is really really old. That would get rid of some of these composer issues: also here, I recommend a conservative composer update nothing

@juicylime

Copy link
Copy Markdown
Author

There also looked to be a conflict with the laminas coding standard library so I went ahead and updated that too. Hopefully that wont pose any issues?

@juicylime

Copy link
Copy Markdown
Author

Ah I found another PR that addresses some of the issues going on here #23. The PR is still open so maybe there's some outstanding issues with the dependencies?

@juicylime

Copy link
Copy Markdown
Author

Looks like laminas-api-tools/api-tools-doctrine only supports until PHP 8.0 so something similar would need to be done there to extend it to 8.1 (If it doesn't break other packages in the process) and the PHP 8.1 tests would pass here once that package has been updated to support 8.1 as well.

I'm not so sure about the PHP 8.0 tests since there are issues with the phpro/zf-doctrine-hydration-module dependency and that only supports up until PHP ^7.0

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

Labels

Enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support for PHP 8.1

2 participants