Add git-runic script

[fredrikekre]

Adds the git-runic script (a modified version of git-clang-format) from #120 (comment) to the repository.

[fredrikekre]

cc @maleadt

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 93.87%. Comparing base (6d9e931) to head (be80940).
Report is 2 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master     #122   +/-   ##
=======================================
  Coverage   93.87%   93.87%           
=======================================
  Files           8        8           
  Lines        3790     3790           
=======================================
  Hits         3558     3558           
  Misses        232      232           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[maleadt]

LGTM! Has been working great locally; thanks for adding the necessary features.

[fredrikekre]

Do you automatically run git runic in a pre-commit hook or similar? Or just manually before committing?

[maleadt]

I personally do it manually, after staging changes, because I want to be able to choose which formatting changes to accept (I know, going against the philosophy of the tool here).

[fredrikekre]

Ha, okay. Just out of curiosity -- what formatting don't you like?

[maleadt]

I generally like the formatting, but sometimes I've carefully laid out values for clarify, e.g.:

• https://github.com/JuliaGPU/CUDA.jl/blob/4e9513b8a4e56629a236b58504d609b1775a8236/src/device/texture.jl#L58-L61
• https://github.com/JuliaGPU/CUDA.jl/blob/4e9513b8a4e56629a236b58504d609b1775a8236/src/device/texture.jl#L169-L172

These become:

@@ -56,31 +56,45 @@ end
 Base.Tuple(x::Vec4) = tuple(x.a, x.b, x.c, x.d)

 for (dispatch_rettyp, julia_rettyp, llvm_rettyp) in
-        ((:Signed,        :(Vec4{Int32}),   :v4u32),
-         (:Unsigned,      :(Vec4{UInt32}),  :v4s32),
-         (:AbstractFloat, :(Vec4{Float32}), :v4f32))
+        (
+        (:Signed, :(Vec4{Int32}), :v4u32),
+        (:Unsigned, :(Vec4{UInt32}), :v4s32),
+        (:AbstractFloat, :(Vec4{Float32}), :v4f32),
+    )

@@ -166,9 +184,13 @@ end
     h0y = h0(fy)
     h1y = h1(fy)

-    vals = g0(fy) .* (g0x .* tex(t, px + h0x, py + h0y) .+
-                      g1x .* tex(t, px + h1x, py + h0y)) .+
-           g1(fy) .* (g0x .* tex(t, px + h0x, py + h1y) .+
-                      g1x .* tex(t, px + h1x, py + h1y))
+    vals = g0(fy) .* (
+        g0x .* tex(t, px + h0x, py + h0y) .+
+            g1x .* tex(t, px + h1x, py + h0y)
+    ) .+
+        g1(fy) .* (
+        g0x .* tex(t, px + h0x, py + h1y) .+
+            g1x .* tex(t, px + h1x, py + h1y)
+    )
     return (unpack(T, vals))
 end

It seems silly to annotate those with # RUNIC NO TOUCHY (or whatever the magical comment is), so I think I prefer the manual approach instead.