From 4dddc3f717765bb2687a6aa576f0513e5fc0cbf8 Mon Sep 17 00:00:00 2001
From: Nutomic <me@nutomic.com>
Date: Sat, 4 Feb 2023 03:32:44 +0900
Subject: [PATCH] Disable pictrs feature in CI to make it faster (#2698)

---
 .drone.yml | 47 +++++++++++++++++++++++++----------------------
 1 file changed, 25 insertions(+), 22 deletions(-)

diff --git a/.drone.yml b/.drone.yml
index 4754aa9c8..8c5fb0d36 100644
--- a/.drone.yml
+++ b/.drone.yml
@@ -29,26 +29,8 @@ steps:
       - rustup component add rustfmt --toolchain nightly
       - cargo +nightly fmt -- --check
 
-
-  - name: cargo clippy
-    image: clux/muslrust:1.67.0
-    environment:
-      CARGO_HOME: .cargo
-    commands:
-        # latest rust for clippy to get extra checks
-        # when adding new clippy lints, make sure to also add them in scripts/fix-clippy.sh
-      - rustup component add clippy
-      - cargo clippy --workspace --tests --all-targets --all-features -- 
-          -D warnings -D deprecated -D clippy::perf -D clippy::complexity 
-          -D clippy::style -D clippy::correctness -D clippy::suspicious
-          -D clippy::dbg_macro -D clippy::inefficient_to_string 
-          -D clippy::items-after-statements -D clippy::implicit_clone 
-          -D clippy::wildcard_imports -D clippy::cast_lossless 
-          -D clippy::manual_string_new -D clippy::redundant_closure_for_method_calls 
-          -D clippy::unused_self
-          -A clippy::uninlined_format_args
-      - cargo clippy --workspace --all-features -- -D clippy::unwrap_used
-
+  # check each package to make sure they compile with default features.
+  # this is required for crates.io
   - name: cargo check
     image: clux/muslrust:1.67.0
     environment:
@@ -64,8 +46,29 @@ steps:
       - cargo check --package lemmy_api_crud
       - cargo check --package lemmy_apub
       - cargo check --package lemmy_routes
-      - cargo check --workspace --no-default-features
-      - cargo check --workspace --all-features
+      - cargo check --workspace
+      - cargo check --workspace --features console
+      # disabled because it takes too long with pict-rs
+      #- cargo check --workspace --all-features
+
+  - name: cargo clippy
+    image: clux/muslrust:1.67.0
+    environment:
+      CARGO_HOME: .cargo
+    commands:
+        # latest rust for clippy to get extra checks
+        # when adding new clippy lints, make sure to also add them in scripts/fix-clippy.sh
+      - rustup component add clippy
+      - cargo clippy --workspace --tests --all-targets --features console --
+          -D warnings -D deprecated -D clippy::perf -D clippy::complexity 
+          -D clippy::style -D clippy::correctness -D clippy::suspicious
+          -D clippy::dbg_macro -D clippy::inefficient_to_string 
+          -D clippy::items-after-statements -D clippy::implicit_clone 
+          -D clippy::wildcard_imports -D clippy::cast_lossless 
+          -D clippy::manual_string_new -D clippy::redundant_closure_for_method_calls 
+          -D clippy::unused_self
+          -A clippy::uninlined_format_args
+      - cargo clippy --workspace --features console -- -D clippy::unwrap_used
 
   - name: lemmy_api_common doesnt depend on diesel
     image: clux/muslrust:1.67.0