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

feat: PHP 8.4 #507

Merged
merged 11 commits into from
Dec 11, 2024
Merged

feat: PHP 8.4 #507

merged 11 commits into from
Dec 11, 2024

Conversation

dkarlovi
Copy link
Contributor

@dkarlovi dkarlovi commented Dec 3, 2024

Closes #506 506

@dkarlovi
Copy link
Contributor Author

dkarlovi commented Dec 3, 2024

@jakzal do you have any idea why this build fails, it works for me locally. 🤔

@jakzal
Copy link
Owner

jakzal commented Dec 3, 2024

@dkarlovi the master branch build started failing couple days ago. Seems to be related to phpunit xsd not being available.

@dkarlovi
Copy link
Contributor Author

dkarlovi commented Dec 3, 2024

@jakzal I've fixed that in this PR actually, my error is related to phive.

@jakzal
Copy link
Owner

jakzal commented Dec 4, 2024

@dkarlovi master is passing now, can you merge or rebase please? Some fixes I applied are similar to yours.

@@ -3,7 +3,7 @@
"description": "Helps to discover and install tools",
"type": "project",
"require": {
"php": "~8.1.0 || ~8.2.0 || ~8.3.0",
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking at this code and the one in devkit, we could extract the supported PHP versions from composer.json here instead of hardcoding it, probably also generate the Github actions YAML file.

@dkarlovi
Copy link
Contributor Author

dkarlovi commented Dec 4, 2024

@jakzal done, but still the same Phive issue, I'm not sure what that's all about.

@dkarlovi
Copy link
Contributor Author

dkarlovi commented Dec 6, 2024

Revisiting this, I have no idea what this is all about, could it be rat limits or something? It works on master, it works for me locally, it fails in this branch. 🤔

@dkarlovi
Copy link
Contributor Author

dkarlovi commented Dec 6, 2024

@jakzal this seems to be due to GH rate limits, if you check the latest build, it says rate limit 60 which is definitely getting consumed by the previous steps, could your runs on master and in your branches somehow use your auth since you're the owner, but mine don't?

Edit: yep, seems that's it, I've updated the build step to use the user-specific secret, it means your pipelines will use your token and mine will use mine IIUC.

@dkarlovi
Copy link
Contributor Author

dkarlovi commented Dec 6, 2024

@jakzal RFR.

@jakzal
Copy link
Owner

jakzal commented Dec 11, 2024

Thank you @dkarlovi 🍻

@jakzal jakzal merged commit 5df45ea into jakzal:master Dec 11, 2024
11 checks passed
@dkarlovi dkarlovi deleted the php84 branch December 11, 2024 11:44
github-actions bot pushed a commit to jakzal/phpqa that referenced this pull request Dec 11, 2024
## Updates

* PHP 8.4 support (thanks @dkarlovi)

* Removed php-coupling-detector

## What's Changed

* Update infection (0.27 -> 0.29) by @jakzal in jakzal/toolbox#508

* feat: PHP 8.4 by @dkarlovi in jakzal/toolbox#507

**Full Changelog**: jakzal/toolbox@v1.84.1...v1.85.0
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.

PHP 8.4
2 participants