alamb commented on PR #7878: URL: https://github.com/apache/arrow-rs/pull/7878#issuecomment-3060156882
> For example, the latest CI run displays errors like: > > > Documenting arrow-ord v55.2.0 (/__w/arrow-rs/arrow-rs/arrow-ord) > > error: unnecessary parentheses around closure body > > --> arrow-ord/src/cmp.rs:276:43 > > | > > 276 | let c = |((l, r), n)| ((l ^ r) | (l & r & n)); > > | ^ ^ > > | > > = note: `-D unused-parens` implied by `-D warnings` > > = help: to override `-D warnings` add `#[allow(unused_parens)]` > > help: remove these parentheses > > | > > 276 - let c = |((l, r), n)| ((l ^ r) | (l & r & n)); > > 276 + let c = |((l, r), n)| (l ^ r) | (l & r & n); > > | > > error: could not document `arrow-ord` Looks to me like the latest nightly rust got more strict and for some reason we build docs with nightly in this crate So to reproduce locally I bet you can do ```shell rustup toolchain install nightly cargo +nightly doc --document-private-items --no-deps --workspace --all-features ``` So in other words it is not related to your PR and CI would likely fail on main if we re-ran it -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: github-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
