ci: add GitHub Actions workflow and fix clippy warnings

Add CI workflow with format, clippy, test, MSRV, and docs jobs.
Fix collapsible if statements in milestones example using let-chains.

Author: pbeets

.github/workflows/ci.yml

@@ -0,0 +1,63 @@
+name: CI
+
+on:
+  push:
+    branches: [main]
+  pull_request:
+    branches: [main]
+
+env:
+  CARGO_TERM_COLOR: always
+
+jobs:
+  fmt:
+    name: Format
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - uses: dtolnay/rust-toolchain@stable
+        with:
+          components: rustfmt
+      - run: cargo fmt --all --check
+
+  clippy:
+    name: Clippy
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - uses: dtolnay/rust-toolchain@stable
+        with:
+          components: clippy
+      - uses: Swatinem/rust-cache@v2
+      - run: cargo clippy --all-targets -- -D warnings
+
+  test:
+    name: Test
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - uses: dtolnay/rust-toolchain@stable
+      - uses: Swatinem/rust-cache@v2
+      - run: cargo test
+
+  msrv:
+    name: MSRV (1.92)
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - uses: dtolnay/rust-toolchain@master
+        with:
+          toolchain: "1.92"
+      - uses: Swatinem/rust-cache@v2
+      - run: cargo build --all-targets
+
+  docs:
+    name: Docs
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - uses: dtolnay/rust-toolchain@stable
+      - uses: Swatinem/rust-cache@v2
+      - run: cargo doc --no-deps
+        env:
+          RUSTDOCFLAGS: -D warnings

examples/milestones.rs

@@ -51,10 +51,10 @@ async fn main() -> Result<(), Box<dyn std::error::Error>> {
         if let Some(start) = &milestone.start_date {
             println!("   Start: {}", start);
         }
-        if let Some(tickers) = &milestone.primary_event_tickers {
-            if !tickers.is_empty() {
-                println!("   Events: {}", tickers.join(", "));
-            }
+        if let Some(tickers) = &milestone.primary_event_tickers
+            && !tickers.is_empty()
+        {
+            println!("   Events: {}", tickers.join(", "));
         }
         println!();
     }
@@ -118,27 +118,26 @@ async fn main() -> Result<(), Box<dyn std::error::Error>> {
                 if let Some(end) = &milestone.end_date {
                     println!("  End:      {}", end);
                 }
-                if let Some(tickers) = &milestone.primary_event_tickers {
-                    if !tickers.is_empty() {
-                        println!("  Primary Events: {}", tickers.join(", "));
-                    }
+                if let Some(tickers) = &milestone.primary_event_tickers
+                    && !tickers.is_empty()
+                {
+                    println!("  Primary Events: {}", tickers.join(", "));
                 }
-                if let Some(tickers) = &milestone.related_event_tickers {
-                    if !tickers.is_empty() {
-                        println!("  Related Events: {}", tickers.join(", "));
-                    }
+                if let Some(tickers) = &milestone.related_event_tickers
+                    && !tickers.is_empty()
+                {
+                    println!("  Related Events: {}", tickers.join(", "));
                 }
                 if let Some(updated) = &milestone.last_updated_ts {
                     println!("  Updated:  {}", updated);
                 }
-                if let Some(details) = &milestone.details {
-                    if let Some(obj) = details.as_object() {
-                        if !obj.is_empty() {
-                            println!("  Details:  {} fields", obj.len());
-                            for (key, value) in obj.iter().take(5) {
-                                println!("    {}: {}", key, value);
-                            }
-                        }
+                if let Some(details) = &milestone.details
+                    && let Some(obj) = details.as_object()
+                    && !obj.is_empty()
+                {
+                    println!("  Details:  {} fields", obj.len());
+                    for (key, value) in obj.iter().take(5) {
+                        println!("    {}: {}", key, value);
                     }
                 }
             }