Skip to content
This repository has been archived by the owner on Feb 18, 2024. It is now read-only.

Added trait TryPush #314

Merged
merged 1 commit into from
Aug 22, 2021
Merged

Added trait TryPush #314

merged 1 commit into from
Aug 22, 2021

Conversation

jorgecarleitao
Copy link
Owner

And implemented it for most of the Mutable API.

@jorgecarleitao jorgecarleitao added the enhancement An improvement to an existing feature label Aug 22, 2021
@codecov
Copy link

codecov bot commented Aug 22, 2021

Codecov Report

Merging #314 (a7f23ca) into main (83256a8) will decrease coverage by 0.01%.
The diff coverage is 76.47%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #314      +/-   ##
==========================================
- Coverage   80.60%   80.59%   -0.02%     
==========================================
  Files         324      324              
  Lines       21255    21268      +13     
==========================================
+ Hits        17132    17140       +8     
- Misses       4123     4128       +5     
Impacted Files Coverage Δ
src/array/boolean/mutable.rs 86.86% <0.00%> (-1.95%) ⬇️
src/array/mod.rs 48.24% <ø> (ø)
src/array/primitive/mutable.rs 85.71% <0.00%> (-1.39%) ⬇️
src/array/utf8/mutable.rs 86.29% <81.25%> (ø)
src/array/binary/mutable.rs 67.94% <82.35%> (ø)
src/array/list/mutable.rs 75.86% <100.00%> (+0.86%) ⬆️
tests/it/array/list/mutable.rs 100.00% <100.00%> (ø)
src/io/json_integration/write.rs 6.25% <0.00%> (+6.25%) ⬆️

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 83256a8...a7f23ca. Read the comment docs.

@jorgecarleitao jorgecarleitao merged commit d278f08 into main Aug 22, 2021
@jorgecarleitao jorgecarleitao deleted the mutable_try_push branch August 22, 2021 19:42
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement An improvement to an existing feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant