Make usage of serde optional behind feature flag (#155)

Author: jpschorr

.github/workflows/ci_build_test.yml

@@ -112,7 +112,7 @@ jobs:
         continue-on-error: true
         run: cargo test --verbose --package partiql-conformance-tests --features "conformance_test" -- -Z unstable-options --format json > cargo_test_results.json
       # Create a conformance report from the `cargo test` json file
-      - run: ./target/debug/generate_cts_report cargo_test_results.json ${GITHUB_SHA} cts_report.json
+      - run: cargo run --features report_tool --bin generate_cts_report cargo_test_results.json ${GITHUB_SHA} cts_report.json
       # Upload conformance report for comparison with future runs
       - name: Upload cts_report.json
         uses: actions/upload-artifact@v3
@@ -164,7 +164,7 @@ jobs:
         if: ${{ steps.download-report.outcome == 'failure' }}
         run: mkdir -p artifact && cp -r ./partiql-conformance-tests/backup_conformance_report.json ./artifact/cts_report.json
       # Run conformance report comparison. Generates cts-comparison-report.md
-      - run: ./target/debug/generate_comparison_report ./artifact/cts_report.json cts_report.json cts-comparison-report.md
+      - run: cargo run --features report_tool --bin generate_comparison_report ./artifact/cts_report.json cts_report.json cts-comparison-report.md
       # Print conformance report to GitHub actions workflow summary page
       - name: print markdown in run
         run: cat cts-comparison-report.md >> $GITHUB_STEP_SUMMARY

.github/workflows/coverage.yml

@@ -23,7 +23,7 @@ jobs:
         with:
           profile: minimal
           # nightly can be very volatile--pin this to a version we know works well...
-          toolchain: nightly-2022-01-10
+          toolchain: nightly-2022-06-11
           override: true
       - name: Cargo Test
         uses: actions-rs/cargo@v1

Cargo.toml

@@ -9,7 +9,11 @@ members = [
   "partiql-eval",
   "partiql-irgen",
   "partiql-parser",
-  "partiql-playground",
   "partiql-rewriter",
+]
+
+exclude = [
+  "partiql-playground",
   "partiql-cli"
 ]
+

partiql-ast/Cargo.toml

@@ -24,8 +24,20 @@ path = "src/lib.rs"
 partiql-source-map = { path = "../partiql-source-map" }
 
 derive_builder = "~0.11.1"
-rust_decimal = "~1.22.0"
-serde = { version = "~1.0.137", features = ["derive"] }
+rust_decimal = { version = "1.25.0", default-features = false, features = ["std"] }
+
+serde = { version = "1.*", features = ["derive"], optional = true }
+
 
 [dev-dependencies]
 
+
+[features]
+default = []
+serde = [
+  "dep:serde",
+  "partiql-source-map/serde",
+  "rust_decimal/serde-with-str",
+  "rust_decimal/serde"
+]
+