chore: standardize environment variables and font installation

- remove unnecessary comment about shell for running on windows
- add environment variables for fonts, version, and extension
- modify font installation loop to use environment variables
- update target operating system condition in renderer.rs

Author: falcucci

.github/workflows/ci.yml

@@ -7,7 +7,12 @@ on:
 
 defaults:
   run:
-    shell: bash # necessary for windows
+    shell: bash
+
+env:
+  FONTS: "Monaspace FiraCode ProFont CascadiaCode Noto"
+  VERSION: "v3.2.1"
+  EXTENSION: ".zip"
 
 jobs:
   fmt:
@@ -116,12 +121,10 @@ jobs:
             brew install font-monaspace-nerd-font
             brew install font-noto-nerd-font
           elif [[ $RUNNER_OS == "Linux" ]]; then
-            VERSION="v3.2.1"
-            EXTENSION=".zip"
             FONT_DIR="${HOME}/.local/share/fonts"
             mkdir -p "$FONT_DIR"
 
-            for font in "${fonts[@]}"; do
+            for font in ${{ env.FONTS }}; do
               ZIP_FILE="${font}${EXTENSION}"
               if [[ "$font" == "Monaspace" ]]; then
                 DOWNLOAD_URL="https://github.com/githubnext/monaspace/releases/download/v1.101/monaspace-v1.101.zip"

src/renderer.rs

@@ -248,7 +248,7 @@ impl Renderer {
             // | Features::TIMESTAMP_QUERY_INSIDE_ENCODERS
         }
 
-        #[cfg(not(any(target_os = "macos", target_os = "linux")))]
+        #[cfg(target_os = "windows")]
         {
             Features::PUSH_CONSTANTS
                 | Features::SPIRV_SHADER_PASSTHROUGH