Skip to content

fix: ensure limit works in elastic#279

Merged
alaeddine-13 merged 1 commit into
mainfrom
fix-limit-find-elastic
Apr 14, 2022
Merged

fix: ensure limit works in elastic#279
alaeddine-13 merged 1 commit into
mainfrom
fix-limit-find-elastic

Conversation

@davidbp

@davidbp davidbp commented Apr 14, 2022

Copy link
Copy Markdown
Contributor

No description provided.

@codecov

codecov Bot commented Apr 14, 2022

Copy link
Copy Markdown

Codecov Report

Merging #279 (e2268b6) into main (d12710c) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #279   +/-   ##
=======================================
  Coverage   86.10%   86.10%           
=======================================
  Files         134      134           
  Lines        6282     6282           
=======================================
  Hits         5409     5409           
  Misses        873      873           
Flag Coverage Δ
docarray 86.10% <100.00%> (ø)

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

Impacted Files Coverage Δ
docarray/array/mixins/find.py 88.60% <100.00%> (ø)

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 d12710c...e2268b6. Read the comment docs.

@alaeddine-13 alaeddine-13 marked this pull request as ready for review April 14, 2022 11:49
@alaeddine-13 alaeddine-13 merged commit e8cc5a7 into main Apr 14, 2022
@alaeddine-13 alaeddine-13 deleted the fix-limit-find-elastic branch April 14, 2022 11:49
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.

2 participants