Skip to content

New api fixes - #2

Merged
SergeyPirogov merged 8 commits into
masterfrom
develop
Mar 23, 2017
Merged

New api fixes#2
SergeyPirogov merged 8 commits into
masterfrom
develop

Conversation

@SergeyPirogov

Copy link
Copy Markdown
Contributor

No description provided.

@pep8speaks

Copy link
Copy Markdown

Hello @SergeyPirogov! Thanks for submitting the PR.

Line 33:80: E501 line too long (87 > 79 characters)
Line 36:80: E501 line too long (81 > 79 characters)
Line 38:80: E501 line too long (80 > 79 characters)
Line 40:80: E501 line too long (82 > 79 characters)
Line 42:80: E501 line too long (89 > 79 characters)

Line 35:80: E501 line too long (81 > 79 characters)

Line 45:80: E501 line too long (96 > 79 characters)
Line 46:80: E501 line too long (97 > 79 characters)

@SergeyPirogov
SergeyPirogov merged commit 90d118e into master Mar 23, 2017
@codecov-io

Copy link
Copy Markdown

Codecov Report

Merging #2 into master will decrease coverage by 0.04%.
The diff coverage is 90.9%.

@@            Coverage Diff             @@
##           master       #2      +/-   ##
==========================================
- Coverage    86.5%   86.45%   -0.05%     
==========================================
  Files          11       11              
  Lines         163      192      +29     
  Branches        6        7       +1     
==========================================
+ Hits          141      166      +25     
- Misses         18       22       +4     
  Partials        4        4
Impacted Files Coverage Δ
testcontainers/postgres.py 100% <100%> (ø) ⬆️
testcontainers/core/generic.py 100% <100%> (ø) ⬆️
testcontainers/mysql.py 100% <100%> (ø) ⬆️
testcontainers/core/container.py 88.46% <77.77%> (-6.42%) ⬇️

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 04a7eee...1d31a52. Read the comment docs.

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.

4 participants