generate schema in workflows (#125)

* generate schema in workflows * fix windows build * compress schema output in release mode
blake-mealey · Feb 1, 2022 · 3a7b2b4 · 3a7b2b4
1 parent b98f56d
commit 3a7b2b4
Show file tree

Hide file tree

Showing 4 changed files with 19 additions and 6 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -30,3 +30,5 @@ jobs:
         run: |
           cargo fmt -- --check
           cargo clippy -- -D warnings
+      - name: Generate schema
+        run: cargo run --bin schema
diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml
@@ -27,7 +27,7 @@ jobs:
         with:
           filename: mantle-win64.zip
           path: target/release/mantle.exe
-      - name: Upload artifacts
+      - name: Upload binary artifact
         uses: actions/upload-artifact@v2
         with:
           name: mantle-win64
@@ -58,7 +58,7 @@ jobs:
         with:
           filename: mantle-macos.zip
           path: target/release/mantle
-      - name: Upload artifacts
+      - name: Upload binary artifact
         uses: actions/upload-artifact@v2
         with:
           name: mantle-macos
@@ -85,24 +85,31 @@ jobs:
         with:
           filename: mantle-linux.zip
           path: target/release/mantle
-      - name: Upload artifacts
+      - name: Upload binary artifact
         uses: actions/upload-artifact@v2
         with:
           name: mantle-linux
           path: mantle-linux.zip
+      - name: Generate schema
+        run: cargo run --bin schema > schema.json
+      - name: Upload schema artifact
+        uses: actions/upload-artifact@v2
+        with:
+          name: schema
+          path: schema.json
   release:
     runs-on: ubuntu-latest
     needs: [windows, macos, linux]
     steps:
       - name: Download binaries
         uses: actions/download-artifact@v2
         with:
-          path: binaries
+          path: artifacts
       - name: Create release
         uses: softprops/action-gh-release@v1
         with:
           # TODO: remove and add body_path once we have an automated release notes generator
           draft: true
           # TODO: remove once we reach a stable version
           prerelease: true
-          files: binaries/*/*
+          files: artifacts/*/*
diff --git a/src/lib/roblox_auth.rs b/src/lib/roblox_auth.rs
@@ -75,7 +75,7 @@ fn get_roblosecurity_from_roblox_studio() -> Option<String> {
         .ok()?;
     let value: String = key.get_value(".ROBLOSECURITY").ok()?;
 
-    parse_roblosecurity_from_roblox_studio(value)
+    parse_roblosecurity_from_roblox_studio(&value)
 }
 
 #[cfg(target_os = "macos")]

diff --git a/src/schema.rs b/src/schema.rs
@@ -53,5 +53,9 @@ fn main() {
         .into_generator()
         .into_root_schema_for::<Config>();
 
+    #[cfg(debug_assertions)]
     println!("{}", serde_json::to_string_pretty(&schema).unwrap());
+
+    #[cfg(not(debug_assertions))]
+    println!("{}", serde_json::to_string(&schema).unwrap());
 }