Address review

Author: daxpedda

.github/workflows/ci.yml

@@ -127,13 +127,8 @@ jobs:
           cargo clippy --target ${{ matrix.target }} -p wgpu --tests --features glsl,spirv
           cargo doc --target ${{ matrix.target }} -p wgpu --no-deps --features glsl,spirv
 
-          # build for WebGL
-          cargo clippy --target ${{ matrix.target }} -p wgpu --tests --features webgl,glsl,spirv
-          cargo doc --target ${{ matrix.target }} -p wgpu --no-deps --features webgl,glsl,spirv
-
           # all features
-          cargo clippy --target ${{ matrix.target }} -p wgpu-hal --tests --all-features
-          cargo clippy --target ${{ matrix.target }} -p wgpu --tests --all-features
+          cargo clippy --target ${{ matrix.target }} -p wgpu -p wgpu-hal --tests --all-features
           cargo doc --target ${{ matrix.target }} -p wgpu --no-deps --all-features
 
       - name: check em
@@ -165,17 +160,10 @@ jobs:
           cargo clippy --target ${{ matrix.target }} -p wgpu -p wgpu-core -p wgpu-info -p player --no-default-features
 
           # Check with all features.
-          # (But watch out for backend-selection features in wgpu-core; some of
-          # those only build on the right platforms.)
-          cargo clippy --target ${{ matrix.target }} -p wgpu-hal --tests --all-features
-          cargo clippy --target ${{ matrix.target }} -p wgpu-core --tests --all-features
-          cargo clippy --target ${{ matrix.target }} -p wgpu -p wgpu-info -p player --tests --all-features
+          cargo clippy --target ${{ matrix.target }} -p wgpu -p wgpu-info -p player -p wgpu-core -p wgpu-hal --tests --all-features
 
           # build docs
-          # (Watch out for backend-selection features in wgpu-core; some of
-          # those only build on the right platforms.)
-          cargo doc --target ${{ matrix.target }} -p wgpu -p wgpu-info -p player --all-features --no-deps
-          cargo doc --target ${{ matrix.target }} -p wgpu-core --no-deps --all-features
+          cargo doc --target ${{ matrix.target }} -p wgpu -p wgpu-info -p player -p wgpu-core -p wgpu-hal --all-features --no-deps
 
   wasm-test:
     name: Test WebAssembly

wgpu-hal/src/dx12/mod.rs

@@ -27,7 +27,7 @@ and [`crate::CommandEncoder::set_render_pipeline`] with
 [`crate::CommandEncoder::set_compute_pipeline`].
 
 For this reason, in order avoid repeating the binding code,
-we are binding everything in [`CommandEncoder::update_root_elements`].
+we are binding everything in `CommandEncoder::update_root_elements`.
 When the pipeline layout is changed, we reset all bindings.
 Otherwise, we pass a range corresponding only to the current bind group.