From a34e0d477eb8bcc9acc5b3df44f843bf636b173a Mon Sep 17 00:00:00 2001 From: Nutomic <me@nutomic.com> Date: Fri, 28 Jul 2023 15:33:23 +0200 Subject: [PATCH] Only run slow CI checks for actual code changes (#3750) * Only run slow CI checks for actual code changes Its a waste of time to compile Rust and run tests when only something like the readme was changed. * also make cache restore/rebuild conditional * remove unused condition --- .woodpecker.yml | 42 ++++++++++++++++++++++++------------------ 1 file changed, 24 insertions(+), 18 deletions(-) diff --git a/.woodpecker.yml b/.woodpecker.yml index 58ab2f0f..6353da5c 100644 --- a/.woodpecker.yml +++ b/.woodpecker.yml @@ -48,6 +48,8 @@ pipeline: - "api_tests/node_modules" secrets: [MINIO_ENDPOINT, MINIO_WRITE_USER, MINIO_WRITE_PASSWORD, MINIO_BUCKET] + when: + path: ["crates", "src", "Cargo.toml", "migrations", "api_tests"] toml_fmt: image: tamasfe/taplo:0.8.1 @@ -65,8 +67,6 @@ pipeline: - rustup toolchain install nightly-2023-07-10 - rustup component add rustfmt --toolchain nightly-2023-07-10 - cargo +nightly-2023-07-10 fmt -- --check - # when: - # platform: linux/amd64 # make sure api builds with default features (used by other crates relying on lemmy api) check_api_common_default_features: @@ -75,8 +75,8 @@ pipeline: CARGO_HOME: .cargo commands: - cargo check --package lemmy_api_common - # when: - # platform: linux/amd64 + when: + path: ["crates", "src", "Cargo.toml", "migrations"] lemmy_api_common_doesnt_depend_on_diesel: image: *muslrust_image @@ -84,8 +84,8 @@ pipeline: CARGO_HOME: .cargo commands: - "! cargo tree -p lemmy_api_common --no-default-features -i diesel" - # when: - # platform: linux/amd64 + when: + path: ["crates", "src", "Cargo.toml", "migrations"] lemmy_api_common_works_with_wasm: image: *muslrust_image @@ -94,6 +94,8 @@ pipeline: commands: - "rustup target add wasm32-unknown-unknown" - "cargo check --target wasm32-unknown-unknown -p lemmy_api_common" + when: + path: ["crates", "src", "Cargo.toml", "migrations"] check_defaults_hjson_updated: image: *muslrust_image @@ -103,8 +105,8 @@ pipeline: - export LEMMY_CONFIG_LOCATION=./config/config.hjson - ./scripts/update_config_defaults.sh config/defaults_current.hjson - diff config/defaults.hjson config/defaults_current.hjson - # when: - # platform: linux/amd64 + when: + path: ["crates", "src", "Cargo.toml", "migrations"] check_diesel_schema: image: willsquire/diesel-cli @@ -115,6 +117,8 @@ pipeline: - diesel migration run - diesel print-schema --config-file=diesel.toml > tmp.schema - diff tmp.schema crates/db_schema/src/schema.rs + when: + path: ["crates", "src", "Cargo.toml", "migrations"] check_diesel_migration_revertable: image: willsquire/diesel-cli @@ -124,6 +128,8 @@ pipeline: commands: - diesel migration run - diesel migration redo + when: + path: ["crates", "src", "Cargo.toml", "migrations"] cargo_clippy: image: *muslrust_image @@ -147,8 +153,8 @@ pipeline: -D clippy::needless_collect -D clippy::unwrap_used -D clippy::indexing_slicing - # when: - # platform: linux/amd64 + when: + path: ["crates", "src", "Cargo.toml", "migrations"] cargo_test: image: *muslrust_image @@ -159,8 +165,8 @@ pipeline: commands: - export LEMMY_CONFIG_LOCATION=../../config/config.hjson - cargo test --workspace --no-fail-fast - # when: - # platform: linux/amd64 + when: + path: ["crates", "src", "Cargo.toml", "migrations"] cargo_build: image: *muslrust_image @@ -169,8 +175,8 @@ pipeline: commands: - cargo build - mv target/x86_64-unknown-linux-musl/debug/lemmy_server target/lemmy_server - # when: - # platform: linux/amd64 + when: + path: ["crates", "src", "Cargo.toml", "migrations", "api_tests"] run_federation_tests: image: node:alpine @@ -183,8 +189,8 @@ pipeline: - cd api_tests/ - yarn - yarn api-test - # when: - # platform: linux/amd64 + when: + path: ["crates", "src", "Cargo.toml", "migrations", "api_tests"] rebuild-cache: image: meltwater/drone-cache:v1 @@ -208,6 +214,8 @@ pipeline: - "api_tests/node_modules" secrets: [MINIO_ENDPOINT, MINIO_WRITE_USER, MINIO_WRITE_PASSWORD, MINIO_BUCKET] + when: + path: ["crates", "src", "Cargo.toml", "migrations", "api_tests"] publish_release_docker: image: woodpeckerci/plugin-docker-buildx @@ -257,5 +265,3 @@ services: environment: POSTGRES_USER: lemmy POSTGRES_PASSWORD: password - # when: - # platform: linux/amd64 -- 2.44.1