-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Would a patchlevel release be possible or is planned already ? #578
Comments
ping. |
Ping. |
@bshaffer You are probably very busy, but could you please tag a new release so that projects that rely on this have a few less errors to worry about when preparing for PHP 8.4 😅 🙏 |
@ajupazhamayil would you be able to help out with this? |
If this helps to anyone: we are running on the commit pin (which seems to work for the time being). "require": {
"firebase/php-jwt": "dev-main#76808fa",
} Just note
|
Since PHP 8.4.1 has been released some days ago, could we expect a It does not show much trust if a framework needs to say:
to announce official PHP8.4 support. |
It seems that the repo is using the release-please bot for releases, and this bot doesn't take |
Damn. Should have made one of the pr commits a |
So would it help if we do a new PR with a proper commit to get it tagged? There's probably something tiny code style thing we can tweak / fix. |
Adds PHP 8.4 to the CI matrix. See firebase#572 and firebase#578
@hafezdivandari @sbuerk I think it would be solved with #583, agreed? |
not sure if |
They usually squash and merge, so the |
This can be closed |
Is there any plan for a next (patchlevel) release ? Could not ind any release schedule.
Specially regarding the PHP8.4 required changes of merged #572 ?
The text was updated successfully, but these errors were encountered: