Set minimum php version to 8.2

[da-mask]

Unfortunately peckphp/peck has a minimum version of php8.3, so could not be included. I could find no other reason not have the minimum version set to 8.2.

[da-mask]

This would close #194

[pushpak1300]

can you please add https://github.com/echolabsdev/prism/blob/main/.github/workflows/tests.yml#L18 php8.2 in test matrix so we can be sure if the code works on php 8.2 ?

[da-mask]

@pushpak1300 Yes, of course, that makes sense. Do you think the other formatting/phpstan workflows should also be set to 8.2 ?

[pushpak1300]

@da-mask I don't think that is needed. Let's wait for @sixlive and see what he thinks ?

[sixlive]

I'm okay with this change, I think we can adopt Laravel minimums. I'll take a closer look this afternoon

[sixlive]

I think we also need to update the formatting GH Action to remove the peck check as well.

[da-mask]

I do have a PR in with Peck to do similar, maybe we give it a few days to see if there is movement over there?

[sixlive]

Yeah, lets hold off a few days and see what happens

[kinsta]

Preview deployments for prism ⚡️

Status	Branch preview	Commit preview
❌ Failed to deploy	N/A	N/A

Commit: a570c52164d8ebef72d11e590ff658f3ff3a3d76

Deployment ID: 99e7007d-95ce-45b3-b36e-660ee5b7d35f

Static site name: prism-97nz9

[da-mask]

I just asked Nuno on his twitch stream about the likelihood of approving a php8.2 PR to Peck, and he didn't seem into it. I've added a commit to remove it from the workflow...

[da-mask]

Hmm, maybe keep holding on this. A maintainer over at Peck has given the thumbs up on my PR there, and suggested it get merged in. That hasn't happened yet, so I would have to re-add Peck back in here. Let me know if you'd prefer me to create a new PR to keep things clean, or happy for me to add another commit bringing back Peck if/when that PR gets merged.