From 3191ff26e14abbfba41a06fd710e5201a8935b7e Mon Sep 17 00:00:00 2001 From: Philipp Schuster Date: Thu, 4 Dec 2025 13:54:09 +0100 Subject: [PATCH] docs: adjust contributing.md: remove need for --all (--workspace) It is no longer needed to add `--all` (which is an alias for `--workspace`). The documentation says "Commands run in the workspace root will run against all workspace members by default" [0]. We however still need `--tests` as this activates the building of tests in `/tests` directories. [0] https://doc.rust-lang.org/cargo/reference/workspaces.html#virtual-workspace Signed-off-by: Philipp Schuster On-behalf-of: SAP philipp.schuster@sap.com --- CONTRIBUTING.md | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index c907d29ee..157098201 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -21,11 +21,11 @@ convention and enforce it through the Continuous Integration (CI) process callin ```sh # We currently rely on nightly-only formatting features -cargo +nightly fmt --all -cargo check --all --all-targets --tests -cargo clippy --all --all-targets --tests +cargo +nightly fmt --all +cargo check --all-targets --tests +cargo clippy --all-targets --tests # Please note that this will not execute integration tests. -cargo test --all --all-targets --tests +cargo test --all-targets --tests # To lint your last three commits gitlint --commits "HEAD~3..HEAD" @@ -49,8 +49,8 @@ have specific features that you regularly develop against. #!/bin/sh cargo +nightly fmt --all -- --check || exit 1 -cargo check --locked --all --all-targets --tests || exit 1 -cargo clippy --locked --all --all-targets --tests -- -D warnings || exit 1 +cargo check --locked --all-targets --tests || exit 1 +cargo clippy --locked --all-targets --tests -- -D warnings || exit 1 ``` You will need to `chmod +x .git/hooks/pre-commit` to have it run on every