Skip to content

Use GitHub actions for continuous integration (CI) - #38

Merged
WyriHaximus merged 2 commits into
reactphp:masterfrom
clue-labs:github-ci
Feb 1, 2021
Merged

Use GitHub actions for continuous integration (CI)#38
WyriHaximus merged 2 commits into
reactphp:masterfrom
clue-labs:github-ci

Conversation

@clue

@clueclue commented Feb 1, 2021

Copy link
Copy Markdown
Member

Bye bye Travis CI, you've served us well.

PHP 8 support requires some minor changes, so I'll file this as a separate PR once this is in.

Builds on top of reactphp/reactphp#442 and #37

Bye bye Travis CI, you've served us well.
@clueclue added this to the v1.6.0 milestone Feb 1, 2021
@clueclue mentioned this pull request Feb 1, 2021
jsor
jsor approved these changes Feb 1, 2021
@clue

clue commented Feb 1, 2021

Copy link
Copy Markdown
MemberAuthor

I've also updated this to include a minor code cleanup, removed any dead code and improved code coverage to 100% similar to reactphp/socket#236:shipit:

@clue
clue requested a review from WyriHaximusFebruary 1, 2021 08:05
jsor
jsor approved these changes Feb 1, 2021
@WyriHaximus
WyriHaximus merged commit 0a225a5 into reactphp:masterFeb 1, 2021
@clue
clue deleted the github-ci branch February 1, 2021 11:41
@SimonFringsSimonFrings mentioned this pull request Nov 15, 2021
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@clue@jsor@WyriHaximus