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

cargo fmt #519

Closed
wants to merge 1 commit into from
Closed

cargo fmt #519

wants to merge 1 commit into from

Conversation

ritchie46
Copy link
Collaborator

A $ cargo fmt run.

@xudong963
Copy link
Contributor

Maybe we should add format check in CI? @jorgecarleitao

@codecov
Copy link

codecov bot commented Oct 10, 2021

Codecov Report

Merging #519 (c31a543) into main (e77d4e1) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #519      +/-   ##
==========================================
- Coverage   80.18%   80.17%   -0.01%     
==========================================
  Files         381      381              
  Lines       23665    23666       +1     
==========================================
- Hits        18975    18974       -1     
- Misses       4690     4692       +2     
Impacted Files Coverage Δ
src/scalar/primitive.rs 81.48% <ø> (ø)
src/array/utf8/ffi.rs 78.94% <100.00%> (+1.16%) ⬆️
src/io/parquet/read/mod.rs 60.00% <100.00%> (ø)
src/io/parquet/read/schema/convert.rs 91.31% <100.00%> (ø)
src/io/parquet/write/fixed_len_bytes.rs 90.69% <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%) ⬇️

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 e77d4e1...c31a543. Read the comment docs.

@jorgecarleitao
Copy link
Owner

I agree, @xudong963 ; would you like to try it?

@ritchie46 ritchie46 mentioned this pull request Oct 10, 2021
@jorgecarleitao
Copy link
Owner

Closing in favor of #521 . Thanks for both, @ritchie46 !

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.

None yet

3 participants