Merge pull request #517 from ChrisRackauckas-Claude/runic-formatting

Switch from JuliaFormatter to Runic.jl for code formatting

Author: ChrisRackauckas

.JuliaFormatter.toml

@@ -1,13 +1,19 @@
-name: "Format Check"
+name: format-check
 
 on:
   push:
     branches:
       - 'master'
+      - 'main'
+      - 'release-'
     tags: '*'
   pull_request:
 
 jobs:
-  format-check:
-    name: "Format Check"
-    uses: "SciML/.github/.github/workflows/format-check.yml@v1"
+  runic:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - uses: fredrikekre/runic-action@v1
+        with:
+          version: '1'

.github/workflows/FormatCheck.yml

@@ -5,14 +5,20 @@ cp("./docs/Project.toml", "./docs/src/assets/Project.toml", force = true)
 
 include("pages.jl")
 
-makedocs(sitename = "RecursiveArrayTools.jl",
+makedocs(
+    sitename = "RecursiveArrayTools.jl",
     authors = "Chris Rackauckas",
     modules = [RecursiveArrayTools],
     clean = true, doctest = false, linkcheck = true,
     warnonly = [:missing_docs],
-    format = Documenter.HTML(assets = ["assets/favicon.ico"],
-        canonical = "https://docs.sciml.ai/RecursiveArrayTools/stable/"),
-    pages = pages)
+    format = Documenter.HTML(
+        assets = ["assets/favicon.ico"],
+        canonical = "https://docs.sciml.ai/RecursiveArrayTools/stable/"
+    ),
+    pages = pages
+)
 
-deploydocs(repo = "github.com/SciML/RecursiveArrayTools.jl.git";
-    push_preview = true)
+deploydocs(
+    repo = "github.com/SciML/RecursiveArrayTools.jl.git";
+    push_preview = true
+)

docs/make.jl

@@ -4,5 +4,5 @@ pages = [
     "Home" => "index.md",
     "AbstractVectorOfArrayInterface.md",
     "array_types.md",
-    "recursive_array_functions.md"
+    "recursive_array_functions.md",
 ]

docs/pages.jl

@@ -5,16 +5,20 @@ using FastBroadcast
 using StaticArraysCore
 
 const AbstractVectorOfSArray = AbstractVectorOfArray{
-    T, N, <:AbstractVector{<:StaticArraysCore.SArray}} where {T, N}
+    T, N, <:AbstractVector{<:StaticArraysCore.SArray},
+} where {T, N}
 
 @inline function FastBroadcast.fast_materialize!(
         ::FastBroadcast.Static.False, ::DB, dst::AbstractVectorOfSArray,
-        bc::Broadcast.Broadcasted{S}) where {S, DB}
+        bc::Broadcast.Broadcasted{S}
+    ) where {S, DB}
     if FastBroadcast.use_fast_broadcast(S)
         for i in 1:length(dst.u)
             unpacked = RecursiveArrayTools.unpack_voa(bc, i)
-            dst.u[i] = StaticArraysCore.similar_type(dst.u[i])(unpacked[j]
-            for j in eachindex(unpacked))
+            dst.u[i] = StaticArraysCore.similar_type(dst.u[i])(
+                unpacked[j]
+                    for j in eachindex(unpacked)
+            )
         end
     else
         Broadcast.materialize!(dst, bc)

ext/RecursiveArrayToolsFastBroadcastExt.jl

@@ -4,7 +4,7 @@ using RecursiveArrayTools
 using ForwardDiff
 
 function ForwardDiff.extract_derivative(::Type{T}, y::AbstractVectorOfArray) where {T}
-    ForwardDiff.extract_derivative.(T, y)
+    return ForwardDiff.extract_derivative.(T, y)
 end
 
 end

ext/RecursiveArrayToolsForwardDiffExt.jl

@@ -3,14 +3,16 @@ module RecursiveArrayToolsMeasurementsExt
 import RecursiveArrayTools
 import Measurements
 
-function RecursiveArrayTools.recursive_unitless_bottom_eltype(a::Type{
-        <:Measurements.Measurement,
-})
-    typeof(oneunit(a))
+function RecursiveArrayTools.recursive_unitless_bottom_eltype(
+        a::Type{
+            <:Measurements.Measurement,
+        }
+    )
+    return typeof(oneunit(a))
 end
 
 function RecursiveArrayTools.recursive_unitless_eltype(a::Type{<:Measurements.Measurement})
-    typeof(oneunit(a))
+    return typeof(oneunit(a))
 end
 
 end

ext/RecursiveArrayToolsMeasurementsExt.jl

@@ -3,16 +3,20 @@ module RecursiveArrayToolsMonteCarloMeasurementsExt
 import RecursiveArrayTools
 import MonteCarloMeasurements
 
-function RecursiveArrayTools.recursive_unitless_bottom_eltype(a::Type{
-        <:MonteCarloMeasurements.Particles,
-})
-    typeof(one(a))
+function RecursiveArrayTools.recursive_unitless_bottom_eltype(
+        a::Type{
+            <:MonteCarloMeasurements.Particles,
+        }
+    )
+    return typeof(one(a))
 end
 
-function RecursiveArrayTools.recursive_unitless_eltype(a::Type{
-        <:MonteCarloMeasurements.Particles,
-})
-    typeof(one(a))
+function RecursiveArrayTools.recursive_unitless_eltype(
+        a::Type{
+            <:MonteCarloMeasurements.Particles,
+        }
+    )
+    return typeof(one(a))
 end
 
 end

ext/RecursiveArrayToolsMonteCarloMeasurementsExt.jl

@@ -13,6 +13,7 @@ function trackedarraycopyto!(dest, src)
             dest.u[i] = slice
         end
     end
+    return
 end
 
 @adjoint function Array(VA::AbstractVectorOfArray{<:ReverseDiff.TrackedReal})
@@ -25,7 +26,8 @@ end
 end
 
 @adjoint function Base.view(
-        A::AbstractVectorOfArray{<:ReverseDiff.TrackedReal, N}, I::Colon...) where {N}
+        A::AbstractVectorOfArray{<:ReverseDiff.TrackedReal, N}, I::Colon...
+    ) where {N}
     view_adjoint = let A = A, I = I
         function (y)
             A = recursivecopy(A)

ext/RecursiveArrayToolsReverseDiffExt.jl

@@ -4,7 +4,8 @@ import SparseArrays
 import RecursiveArrayTools
 
 function Base.copyto!(
-        dest::SparseArrays.AbstractCompressedVector, A::RecursiveArrayTools.ArrayPartition)
+        dest::SparseArrays.AbstractCompressedVector, A::RecursiveArrayTools.ArrayPartition
+    )
     @assert length(dest) == length(A)
     cur = 1
     @inbounds for i in 1:length(A.x)
@@ -15,7 +16,7 @@ function Base.copyto!(
         end
         cur += length(A.x[i])
     end
-    dest
+    return dest
 end
 
 # Fix for issue #486: Define issparse for AbstractVectorOfArray