search

cebe / php-openapi

Read and write OpenAPI yaml/json files and make the content accessible in PHP objects.
MIT License
464 stars 87 forks source link

[openapi v3.1] PHP 8.2 deprecations #178

Open WyriHaximus opened 1 year ago

rogervila commented 1 year ago

@cebe LGTM :)

pieterocp commented 1 year ago

Suggestion: We might want to add Php 8.2 to .github/workflows/php.yml to reduce the risk of regression going forward.

WyriHaximus commented 1 year ago

@pieterocp Whoops good point, I sometimes forget not everyone automated dealing with that out of their mind 😅 . Will either update this PR or file a PR to introduce that. (But probably both.)

DEVizzent commented 1 year ago

@WyriHaximus As cebe seems don't manage this repository from long time ago, and I have created and published a fork of this library. So if you want to use this changes you can send the PR to https://github.com/DEVizzent/cebe-php-openapi

WyriHaximus commented 1 year ago

@DEVizzent Sure, did you also pull in the other PR's in I have open here? Looks like you also kept the namespaces the same so it's usable as, hopefully, a temporary drop in until @cebe has time for this package again :+1:

DEVizzent commented 1 year ago

@WyriHaximus No sorry I didn't pull them. If you want to do it it will be great, if not I will do it next weekend.

WyriHaximus commented 1 year ago

@DEVizzent Sure will do :+1:

WyriHaximus commented 1 year ago

@DEVizzent Any plans with the clone you made? Got the PR's in there as you requested but don't seen any activity on them yet

DEVizzent commented 1 year ago

@WyriHaximus sorry, I didn't get any notification. I will take a look tomorrow. I use to spend time on Open Software all Thursday.

WyriHaximus commented 1 year ago

@DEVizzent Doh yeah then it all makes sense :joy: .