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

Dependencies update #1345

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

aldanor
Copy link
Contributor

@aldanor aldanor commented Dec 24, 2022

There's a few dependencies that are quite outdated (including some hefty ones, like zstd). The only one left outdated is odbc-api since I'm unsure about that.

@codecov
Copy link

codecov bot commented Dec 24, 2022

Codecov Report

Base: 83.17% // Head: 83.14% // Decreases project coverage by -0.02% ⚠️

Coverage data is based on head (7bcecb2) compared to base (2101c8e).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1345      +/-   ##
==========================================
- Coverage   83.17%   83.14%   -0.03%     
==========================================
  Files         370      370              
  Lines       40191    40135      -56     
==========================================
- Hits        33427    33370      -57     
- Misses       6764     6765       +1     
Impacted Files Coverage Δ
src/compute/aggregate/sum.rs 77.08% <100.00%> (-6.67%) ⬇️
src/compute/hash.rs 100.00% <100.00%> (ø)
src/io/ipc/read/array/boolean.rs 90.74% <0.00%> (-7.41%) ⬇️
src/io/ipc/read/file.rs 96.87% <0.00%> (+0.44%) ⬆️

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

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@aldanor aldanor force-pushed the update-deps-20221224 branch from 63407db to 7bcecb2 Compare December 24, 2022 13:05
@jorgecarleitao
Copy link
Owner

This seems to stop the CI from working for wasm32. Any idea why?

@aldanor
Copy link
Contributor Author

aldanor commented Jan 5, 2023

@jorgecarleitao I think I know and I'll fix - but could you please force-rerun the CI jobs first? (I don't have permissions)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants