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

Upgrade to php 8.1 and psr/log 3 #297

Merged
merged 3 commits into from Aug 22, 2023
Merged

Conversation

mfadul24
Copy link
Contributor

No description provided.

Copy link

@marcwitteveen marcwitteveen left a comment

Choose a reason for hiding this comment

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

Changes are needed for a sucsesfull installation

@mfadul24
Copy link
Contributor Author

@marcwitteveen Did you mean the conflicts or the usage?

I've update the PR to include latest changes on master.

saadehmad added a commit to KuickApps/infusionsoft-php that referenced this pull request Oct 17, 2022
@mfadul24
Copy link
Contributor Author

mfadul24 commented Dec 2, 2022

@ROMzombie Any possibility to get this merged?

@hamzaali00001
Copy link

Any update on this? Can't use this library on Laravel 10.0 as it requires monolog v3 which requires psr/log 2.0|3.0

Is it possible that you could bump psr/log?

Thanks.

@mfadul24
Copy link
Contributor Author

Updated with current master. Is there any way to get this approved?

@mnordstrom
Copy link

Squeaky wheel gets the grease. Let's get these changes approved!

@benleivian
Copy link

+1 thank you for working to get php8.1 support!

@silverstein
Copy link

Can we please get this approved? 🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏🙏 @ROMzombie

@ROMzombie ROMzombie self-requested a review August 21, 2023 22:49
@mnordstrom
Copy link

Thank you for the approvals @ROMzombie and @bbooth!!

@ROMzombie ROMzombie merged commit 975ea80 into infusionsoft:master Aug 22, 2023
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.

None yet