Skip to content
This repository was archived by the owner on Feb 10, 2023. It is now read-only.

Run unit tests on github actions (backport)#41

Merged
alexey-milovidov merged 11 commits intoclickhousefrom
ci
Jul 13, 2021
Merged

Run unit tests on github actions (backport)#41
alexey-milovidov merged 11 commits intoclickhousefrom
ci

Conversation

@azat
Copy link
Copy Markdown
Member

@azat azat commented Jul 13, 2021

Currently fork of Poco for ClickHouse has lots of custom patches, and those patches sometimes covered by some test in ClickHouse, but nobody runs unit tests, let's fix this.

@azat azat force-pushed the ci branch 2 times, most recently from 05209a3 to a24242c Compare July 13, 2021 07:47
@azat azat force-pushed the ci branch 2 times, most recently from 942b6ba to 89b8af4 Compare July 13, 2021 09:01
@azat azat marked this pull request as draft July 13, 2021 09:09
@azat azat force-pushed the ci branch 2 times, most recently from 6aeaa86 to b11c6eb Compare July 13, 2021 18:13
@azat azat changed the title Backport github actions Run unit tests on github actions (backport) Jul 13, 2021
azat added 10 commits July 13, 2021 22:15
v2: disable macos/windows/crossbuild builds for now
v3: use unbundled build
v4: use bundled expat
v5: disable autotools builds (sanitizers looks useless anyway since
    SANITIZEFLAGS does not passed through)

Based-on: 1a1018d ("add GitHub workflow")
seems that appinf.com drops ICMP packets, so replace it with
one.one.one.one
@azat azat marked this pull request as ready for review July 13, 2021 19:46
@azat azat requested a review from alexey-milovidov July 13, 2021 19:53
@alexey-milovidov alexey-milovidov merged commit 44bed72 into clickhouse Jul 13, 2021
@alexey-milovidov alexey-milovidov self-assigned this Jul 13, 2021
@azat azat deleted the ci branch July 13, 2021 20:30
@azat azat mentioned this pull request Jun 30, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants