Skip to content

test: ci fix#520

Merged
hanxiao merged 3 commits into
mainfrom
test-ci-fix
Sep 3, 2022
Merged

test: ci fix#520
hanxiao merged 3 commits into
mainfrom
test-ci-fix

Conversation

@alaeddine-13

Copy link
Copy Markdown
Member

Goals:

@codecov

codecov Bot commented Sep 2, 2022

Copy link
Copy Markdown

Codecov Report

Merging #520 (a24df28) into main (0b2326b) will decrease coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #520      +/-   ##
==========================================
- Coverage   84.10%   84.05%   -0.05%     
==========================================
  Files         142      142              
  Lines        7013     7013              
==========================================
- Hits         5898     5895       -3     
- Misses       1115     1118       +3     
Flag Coverage Δ
docarray 84.05% <ø> (-0.05%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
docarray/array/mixins/embed.py 10.52% <0.00%> (-78.95%) ⬇️
docarray/array/mixins/post.py 18.60% <0.00%> (-74.42%) ⬇️
docarray/array/mixins/group.py 69.23% <0.00%> (-17.95%) ⬇️
docarray/helper.py 68.77% <0.00%> (-12.67%) ⬇️
docarray/document/mixins/sugar.py 56.00% <0.00%> (-12.00%) ⬇️
docarray/array/mixins/traverse.py 90.90% <0.00%> (-4.14%) ⬇️
docarray/array/mixins/setitem.py 81.96% <0.00%> (+0.81%) ⬆️
docarray/array/storage/redis/getsetdel.py 96.77% <0.00%> (+1.61%) ⬆️
docarray/array/storage/elastic/getsetdel.py 100.00% <0.00%> (+1.69%) ⬆️
docarray/array/storage/sqlite/seqlike.py 84.31% <0.00%> (+1.96%) ⬆️
... and 14 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@alaeddine-13 alaeddine-13 changed the title Test ci fix test: ci fix Sep 2, 2022
Comment thread tests/unit/document/test_converters.py
@hanxiao hanxiao merged commit b268f6f into main Sep 3, 2022
@hanxiao hanxiao deleted the test-ci-fix branch September 3, 2022 15:20
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