Skip cargo-about in CI to save 5 minutes (#409)

.github/workflows/ci.yml

@@ -23,28 +23,42 @@ jobs:
         node-version: '16.x'
 
     - name: 🚧 Install Node dependencies
-      run: cd frontend && npm ci
+      run: |
+        cd frontend
+        npm ci
 
     - name: 🔼 Update Rust to latest stable
-      run: rustc --version && rustup update stable && rustc --version
+      run: |
-
+        rustc --version
-    - name: 📦 Install cargo-about
+        rustup update stable
-      run: cargo install cargo-about
+        rustc --version
 
     - name: 🌐 Build Graphite web code
-      run: cd frontend && npm run build
+      env:
+        NODE_ENV: production
+        SKIP_CARGO_ABOUT: true
+      run: |
+        cd frontend
+        npm run build
 
     - name: 👕 Lint Graphite web formatting
-      run: export NODE_ENV=production && cd frontend && npm run lint
+      env:
+        NODE_ENV: production
+      run: |
+        cd frontend
+        npm run lint
 
     - name: 🔬 Check Rust formatting
-      run: cargo fmt --all -- --check
+      run: |
+        cargo fmt --all -- --check
 
     - name: 🦀 Build Rust code
-      run: cargo build --verbose
+      run: |
+        cargo build --verbose
 
     - name: 🧪 Run Rust tests
-      run: cargo test --verbose
+      run: |
+        cargo test --verbose
 
   cargo-deny:
     runs-on: ubuntu-latest

frontend/.eslintrc.js

@@ -59,8 +59,8 @@ module.exports = {
 		"linebreak-style": ["error", "unix"],
 		"eol-last": ["error", "always"],
 		"max-len": ["error", { code: 200, tabWidth: 4 }],
-		"no-console": process.env.NODE_ENV === "production" ? "warn" : "off",
+		"no-console": "warn",
-		"no-debugger": process.env.NODE_ENV === "production" ? "warn" : "off",
+		"no-debugger": "warn",
 		"no-param-reassign": ["error", { props: false }],
 		"no-bitwise": "off",
 		"no-shadow": "off",

frontend/vue.config.js

@@ -105,7 +105,7 @@ module.exports = {
 
 function formatThirdPartyLicenses(jsLicenses) {
 	let rustLicenses = null;
-	if (process.env.NODE_ENV === "production") {
+	if (process.env.NODE_ENV === "production" && process.env.SKIP_CARGO_ABOUT === undefined) {
 		try {
 			rustLicenses = generateRustLicenses();
 		} catch (e) {