Rust / Check Cargo.toml formatting (amd64, stable) (pull_request) Failing

Question

Rust / Check Cargo.toml formatting (amd64, stable) (pull_request) Failing

jiangzhx opened this issue a year ago · comments

zhenxing jiang commented a year ago

Describe the bug
Rust / Check Cargo.toml formatting (amd64, stable) (pull_request) Failing

To Reproduce
submit a pr.

Dan Harris · Answer 1 · Fri Feb 10 2023 01:07:58 GMT+0800 (China Standard Time)

FYI Fixed this in #658. If that PR cannot be merged shortly (it's rather large) then we can pull into a separate one

zhenxing jiang · Answer 2 · Fri Feb 10 2023 11:09:19 GMT+0800 (China Standard Time)

FYI Fixed this in #658. If that PR cannot be merged shortly (it's rather large) then we can pull into a separate one

it's my pr #661 .

still got error on github action https://github.com/apache/arrow-ballista/actions/runs/4140311128/jobs/7159094110#step:8:1

Dan Harris · Answer 3 · Fri Feb 10 2023 18:31:24 GMT+0800 (China Standard Time)

FYI Fixed this in #658. If that PR cannot be merged shortly (it's rather large) then we can pull into a separate one

it's my pr #661 .

still got error on github action https://github.com/apache/arrow-ballista/actions/runs/4140311128/jobs/7159094110#step:8:1

Right, #658 would have to be merged or you can just cherry-pick the commit with the fix.

Andy Grove · Answer 4 · Fri Feb 10 2023 22:38:20 GMT+0800 (China Standard Time)

Just fyi, DataFusion had the same issue and resolved it: apache/datafusion#5180

zhenxing jiang · Answer 5 · Mon Feb 13 2023 12:10:02 GMT+0800 (China Standard Time)

thanks everyone,it's fixed!