abnobdoss opened a new pull request, #2771: URL: https://github.com/apache/iceberg-rust/pull/2771
## Which issue does this PR close? - None. Implements the `check` job split from discussion #2753. ## What changes are included in this PR? Splits the old `check` job (ubuntu + macos) into two ubuntu jobs: - `lint`: the non-compiling checks (license header, toml format, cargo fmt, Cargo.lock, cargo-machete, typos). The standalone Typos workflow is folded in, and taplo and cargo-machete come as prebuilt binaries. - `clippy`: `make check-clippy` only. As suggested in #2753, the heavier jobs now have `needs: lint`, so a fmt or typo mistake fails fast without spinning up builds. The tradeoff is that a lint failure blocks all downstream jobs. This also removes the macOS clippy run. Nothing in the repo is gated on `target_os`, so clippy output is identical across platforms, and macOS compile coverage stays in the build jobs. The macOS `check` job was CI's slowest at 10 to 12 minutes. ## Are these changes tested? CI-only change, exercised by this PR's own CI run. -- 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]
