-
Notifications
You must be signed in to change notification settings - Fork 18
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
[WIP] Update dependencies #31
Conversation
Very nice, could you also make the tests / style checks run on GitHub actions? You can copy some stuff from our packages: https://github.com/spatie/laravel-backup/tree/main/.github/workflows |
I will look into this in the coming days. |
Would be good to have this on php 8.1 |
I think everything must be in order now. As far as I can see at least the PHP CS Fixer has ran. I'm not entirely sure if all is OK now. Can you check? |
Seems ok when quickly glancing over the code. Did you test out the CLI tool itself? Is it working? |
It is:
|
Thank you for your work on this! |
Hi,
We were using this package but it wasn't working anymore since at least PHP 8.1, I took the liberty updating the dependencies.
Let me know if there are any issues and/or if you want to merge this.
Thank you 😄