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

ci: github actions #99

Merged
merged 1 commit into from
Feb 19, 2024
Merged

ci: github actions #99

merged 1 commit into from
Feb 19, 2024

Conversation

soullivaneuh
Copy link
Collaborator

Summary

Travis CI replacement.

Details: #98 (comment)

@soullivaneuh soullivaneuh marked this pull request as ready for review February 19, 2024 11:03
@soullivaneuh soullivaneuh force-pushed the ci/github-actions branch 5 times, most recently from e7e8e58 to f741993 Compare February 19, 2024 15:11
@soullivaneuh
Copy link
Collaborator Author

@Algatux I did a setup with a matrix similar to the old Travis CI configuration and we have a lot of failures... do you remember the state of the previous Travis CI run? 🤔

@soullivaneuh soullivaneuh force-pushed the ci/github-actions branch 4 times, most recently from 930717d to 4a853e6 Compare February 19, 2024 16:26
@soullivaneuh
Copy link
Collaborator Author

soullivaneuh commented Feb 19, 2024

@Algatux I did find a way to make it working, but I had to remove the SF v3 support. I think it is acceptable according to the age of this release... Also it respect the semver rules What do you think?

@soullivaneuh
Copy link
Collaborator Author

I'll start with that to start the PHP8 compatibility work. It may be adjusted in the meantime.

@soullivaneuh soullivaneuh merged commit cef5622 into master Feb 19, 2024
11 checks passed
@soullivaneuh soullivaneuh deleted the ci/github-actions branch February 19, 2024 17:00
@Algatux
Copy link
Owner

Algatux commented Feb 23, 2024

@soullivaneuh sorry to be late, thanks for all the work you are doing. Anyway yep I agree with the removal of symfony v3 as a supported version

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.

2 participants