@cebe LGTM :)

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

@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.)

@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

@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 👍

@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.

@DEVizzent Sure will do 👍

@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

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

@DEVizzent Doh yeah then it all makes sense 😂 .

pipeline on master is currently running against php 8.1, 8.2 and 8.3, is this still relevant?

I think this was implemented in
(Nov 14, 2024) 2ebe618

pipeline on master is currently running against php 8.1, 8.2 and 8.3, is this still relevant?

No clue, will have a look some time this weekend.

I think this was implemented in (Nov 14, 2024) 2ebe618

Huh? That change is about yaml, the code in this PR is PHP

Yes, that is true, but the PHP code changed earlier.
I haven't check if all the modifications from this PR are already in the master or not. I just check some of them.

https://github.com/cebe/php-openapi/blob/master/src/spec/Paths.php#L186