andygrove opened a new pull request, #2304:
URL: https://github.com/apache/datafusion-ballista/pull/2304
# Which issue does this PR close?
None. `main` is currently red and this unblocks it.
# Rationale for this change
`ci/scripts/rust_fmt.sh` (`cargo fmt --all -- --check`) fails on `main` as
of 1fe9e6f70 (#2244, merged today), so the `Lint` job fails on every open pull
request — CI checks the PR merged with `main`, so a PR that has not touched
this file still inherits the failure.
`cargo fmt --all -- --check` on `main` reports one violation:
```
Diff in chaos-testing/src/bin/chaos-executor.rs:74
```
That is the only one in the workspace.
The formatting is version-independent: rustfmt 1.97.0 and 1.97.1 both reject
the current text and both produce the replacement in this PR, so this is not
toolchain drift on the runner.
# What changes are included in this PR?
The output of `cargo fmt --all`, which is limited to the `health_server`
binding added by #2244. The chained `.ok().map(|port| { ... })` moves onto its
own lines and the `unwrap_or_else` closure wraps at `max_width = 90`.
Formatting only. No behavior change.
# Are there any user-facing changes?
No.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
