Better PHP Support

[HighLiuk]

Would close #2432.

Adds several options & revisits how validation works.
Also, it adds a "PHP Version" option to target a specific version of PHP, turning off features that were not supported in that version automatically. More on this on the related issue.

[HighLiuk]

@dvdsgl here we are. Question: how is Quicktype supposed to run test against several languages specific code? I'd like to add PHP specific tests (parsing would be nice to start with, but would be nice to run PHP tests on the generated classes if that's possible).

[dvdsgl]

This is amazing!

Our CI already runs PHP tests.

Take a look at test/languages.ts – search for PHP

[dvdsgl]

Here's the PHP CI test run: https://github.com/glideapps/quicktype/actions/runs/7494968972/job/20406252954?pr=2470

[HighLiuk]

Thanks @dvdsgl I'll check them out and update the PR as soon as more tests pass

By the way, tests are running on PHP 8.2 in CI, but if I target different PHP versions, it would be nicer if I can run tests for each PHP version.
Also, last PHP version is 8.3 since a couple weeks

[dvdsgl]

Sure thing, we can configure the system to run on multiple versions of PHP. I'll just warn you right now that our CI is in a weird state.

[dvdsgl]

Hi there. Our CI is now fixed – please rebase.

[dvdsgl]

@HighLiuk I'd love to merge this—can you rebase so I can run the CI tests?

[HighLiuk]

Hey @dvdsgl the fact is that I've found on my machine that tests don't pass. So I was diving deeper and then just had other things to do and not so much time to make the tests pass... though I know it's a requirement.

Anyway let's check them out together, I've rebased it (I guess)

[dvdsgl]

Oh! Sorry, I thought the tests were passing in CI and we just needed to update before merge. Thank you for updating anyway.

…

On Mon, Apr 15, 2024 at 3:55 AM Luca Di Fazio ***@***.***> wrote: Hey @dvdsgl <https://github.com/dvdsgl> the fact is that I've found on my machine that tests don't pass. So I was diving deeper and then just had other things to do and not so much time to make the tests pass... though I know it's a requirement. — Reply to this email directly, view it on GitHub <#2470 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAA2NJKUHYEQNBKJBX53TZTY5OBW5AVCNFSM6AAAAABBXF2PTSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANJWGAYDGNRSGM> . You are receiving this because you were mentioned.Message ID: ***@***.***>

[HighLiuk]

Hey @dvdsgl, do you have some easy trick to easily test changes in local development (like Nodemon/Vite), without building all the way down every single time I make a change?

[SimonCahill]

Any updates on this? PHP support is practically unusable in its current state...

[HighLiuk]

Sorry I have no time to work on this. If anyone wants to fork to follow up I'd be glad

[SimonCahill]

I doubt that, even if someone were able to fork and continue your work, it would be merged.
This repo seems to be dying.

[schani]

This PR has merge conflicts with master, and "allow edits from maintainers" is not enabled on the fork, so we can't update the branch for you. Could you merge master (or rebase) and resolve the conflicts? Enabling maintainer edits would also let us help with future conflicts. Thanks!