Skip to content

Conversation

@yakimka
Copy link
Contributor

@yakimka yakimka commented Jan 11, 2022

Added a testcontainer for the ClickHouse database.

@yakimka yakimka requested a review from tillahoffmann March 26, 2022 20:20
@codecov-commenter
Copy link

codecov-commenter commented Mar 28, 2022

Codecov Report

Merging #173 (2bd1eaf) into master (1f0ca14) will increase coverage by 0.51%.
The diff coverage is 100.00%.

❗ Current head 2bd1eaf differs from pull request most recent head 5f589e8. Consider uploading reports for the commit 5f589e8 to get more accurate results

@@            Coverage Diff             @@
##           master     #173      +/-   ##
==========================================
+ Coverage   85.08%   85.60%   +0.51%     
==========================================
  Files          24       25       +1     
  Lines         637      653      +16     
  Branches       61       60       -1     
==========================================
+ Hits          542      559      +17     
  Misses         75       75              
+ Partials       20       19       -1     
Impacted Files Coverage Δ
testcontainers/clickhouse.py 100.00% <100.00%> (ø)
testcontainers/compose.py 100.00% <0.00%> (+1.75%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 066b178...5f589e8. Read the comment docs.

@yakimka
Copy link
Contributor Author

yakimka commented Mar 31, 2022

@tillahoffmann can you review this, please?

Copy link
Contributor

@tillahoffmann tillahoffmann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Two small questions, but looks great otherwise!

@yakimka yakimka requested a review from tillahoffmann April 2, 2022 12:23
@tillahoffmann tillahoffmann merged commit 2aefee2 into testcontainers:master Apr 3, 2022
@yakimka yakimka deleted the clickhouse branch April 3, 2022 18:50
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.

3 participants