Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

support php 8.4 #45

Merged
merged 1 commit into from
Dec 9, 2024
Merged

support php 8.4 #45

merged 1 commit into from
Dec 9, 2024

Conversation

fideloper
Copy link
Collaborator

No description provided.

@KTanAug21
Copy link
Contributor

KTanAug21 commented Dec 9, 2024

Looks great, thank you @fideloper for adding this Pull Request to include PHP 8.4 compatibility!

@rubys can you please check if this can be merged( thank you in advance! )?
Here is how it basically goes:

  1. flyctl checks php version from composer.json or the php -v command

  2. flyctl calls the dockerfile-laravel and passes the PHP version

  3. dockerfile-laravel generates the same Dockerfile and required files needed based on scan

  4. The dockerfile generated uses the PHP version identified by flyctl, and uses it to use the proper php config file!

  5. I've also run ./vendor/bin/pest locally and everything passes

@fideloper fideloper merged commit 762b43f into main Dec 9, 2024
2 checks passed
@fideloper fideloper deleted the support-php84 branch December 9, 2024 16:00
@fideloper
Copy link
Collaborator Author

Sorry I was premature there. This won't go live to anyone until a release is made, however.

@rubys
Copy link
Contributor

rubys commented Dec 10, 2024

What are the steps for making a release?

@fideloper
Copy link
Collaborator Author

I believe they are

  1. make/push a tag
  2. make a release from github’s release page within the repo

I use their magic AI button that creates the release notes.

@rubys
Copy link
Contributor

rubys commented Dec 12, 2024

Oh, so unlike Ruby and Node and others that have a separate repository, composer can use github as it's repository. TIL. Sweet!

@fideloper
Copy link
Collaborator Author

fideloper commented Dec 13, 2024 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants