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

Made try_push_valid public again #509

Merged
merged 1 commit into from Oct 9, 2021

Conversation

ritchie46
Copy link
Collaborator

This adds a pub to fn MutableListArray::try_push_valid. This was already public in previous releases. Maybe it got private due to some trait refactors?

@ritchie46 ritchie46 changed the title pub try_push_valid pub try_push_valid Oct 8, 2021
@codecov
Copy link

codecov bot commented Oct 8, 2021

Codecov Report

Merging #509 (c83e61e) into main (8a7f1d7) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #509      +/-   ##
==========================================
- Coverage   80.17%   80.17%   -0.01%     
==========================================
  Files         381      381              
  Lines       23659    23659              
==========================================
- Hits        18969    18968       -1     
- Misses       4690     4691       +1     
Impacted Files Coverage Δ
src/array/list/mutable.rs 66.33% <100.00%> (ø)
src/compute/arithmetics/time.rs 44.89% <0.00%> (-2.05%) ⬇️
src/io/avro/read/schema.rs 53.57% <0.00%> (-1.20%) ⬇️
src/io/ipc/write/common.rs 76.28% <0.00%> (+0.64%) ⬆️

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 8a7f1d7...c83e61e. Read the comment docs.

@jorgecarleitao jorgecarleitao added the bug Something isn't working label Oct 9, 2021
@jorgecarleitao jorgecarleitao changed the title pub try_push_valid Made try_push_valid public again Oct 9, 2021
@jorgecarleitao jorgecarleitao merged commit 0987215 into jorgecarleitao:main Oct 9, 2021
@jorgecarleitao
Copy link
Owner

Released as v0.6.2 :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants