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

php 8.0 support and moving CI to github actions #534

Closed
wants to merge 6 commits into from

Conversation

cbl
Copy link
Contributor

@cbl cbl commented Nov 29, 2020

This pr is an extension to #533. As discussed I moved the CI from travis to github actions.

A completed run can be found in my fork.

All 6 tests where completed in 54s:
Bildschirmfoto 2020-11-29 um 10 05 00

cbl added 6 commits November 27, 2020 21:11

Verified

This commit was signed with the committer’s verified signature.
driesvints Dries Vints
@cbl
Copy link
Contributor Author

cbl commented Nov 29, 2020

@cviebrock I see that you are less active and I was wondering if you would like some help, maintaining this package. I use the package in my framework litstack that we use in the company I work for in many projects. Of course we always like to keep the dev environment up to date. Therefore it is important for us to be able to apply updates quickly. We have already thought about creating our own sluggable package to have it in our hands, however you have done a pretty good job. So I wonder if you would be open to add me as maintainer so I can prepare it for updates even before the release.

@cviebrock
Copy link
Owner

Thanks for showing me how this is done with Github actions! I will probably migrate over to this soon ... just keeping that change separate from the PHP 8 support (which is now merged).

Thanks again!

@cviebrock
Copy link
Owner

Closing this as the package now has PHP 8.0 support and uses Github actions for CI. Thanks!

@cviebrock cviebrock closed this Jan 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants