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: Complete the CI #1324

Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Open

Conversation

theofidry
Copy link
Contributor

@theofidry theofidry commented Feb 4, 2023

This is a draft to show a bit of what I would like to see added in the CI:

  • A more complete matrix to run the tests on
  • A test build for the PHAR

@theofidry theofidry changed the title Draft: ci: Complete the CI ci: Complete the CI Oct 26, 2023
@theofidry theofidry marked this pull request as ready for review October 26, 2023 08:48
@theofidry
Copy link
Contributor Author

@peter279k should now be ready to review. Note that there is one failing test highlighted by the job, but in essence it already exists and should be fixed separately IMO.

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

1 participant