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

Remove 404 Travis image #15714

Merged
merged 4 commits into from
Sep 3, 2024
Merged

Remove 404 Travis image #15714

merged 4 commits into from
Sep 3, 2024

Conversation

petk
Copy link
Member

@petk petk commented Sep 2, 2024

Travis was suspended #15314

[skip ci]

Travis was suspended php#15314

[skip ci]
@iluuu1994
Copy link
Member

Travis sent me an e-mail a week ago or so that they're aware of the issue and working on it. Not sure what that means in terms of the timeline. The build isn't crucial to us, so I don't object to dropping it. In that case, we should drop all related files though.

@petk
Copy link
Member Author

petk commented Sep 2, 2024

I've appended two commits here then. First removes the Travis related files and some documentation mentions, and the second removes travis related functionality from tests. Should we go with entire removal here or only files-related removal? Otherwise, Travis can be theoretically re-enabled back if such need comes by looking at this commit(s) or the PHP-8.3 branch.

@iluuu1994
Copy link
Member

iluuu1994 commented Sep 3, 2024

I don't object to full removal. Travis is one of the more unreliable CI providers. Whether to remove 8.2+ doesn't matter much to me.

run-tests.php Outdated Show resolved Hide resolved
Co-authored-by: Gina Peter Banyard <[email protected]>
@petk petk merged commit 13f0411 into php:master Sep 3, 2024
10 of 11 checks passed
@petk petk deleted the patch-README branch September 3, 2024 23:15
@petk
Copy link
Member Author

petk commented Sep 3, 2024

Merged to master branch then. Thanks everyone.

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.

4 participants