Apply suggestions from code review

Author: mtfishman

src/factorizations/svd.jl

@@ -198,7 +198,7 @@ true
 ```
 """
 function svd(A; kwargs...)
-  SVD(svd!(eigencopy_oftype(A, LinearAlgebra.eigtype(eltype(A))); kwargs...))
+  return SVD(svd!(eigencopy_oftype(A, LinearAlgebra.eigtype(eltype(A))); kwargs...))
 end
 
 LinearAlgebra.svdvals(usv::SVD{<:Any,T}) where {T} = (usv.S)::AbstractVector{T}

test/runtests.jl

@@ -15,11 +15,11 @@ const GROUP = uppercase(
 
 "match files of the form `test_*.jl`, but exclude `*setup*.jl`"
 function istestfile(fn)
-  endswith(fn, ".jl") && startswith(basename(fn), "test_") && !contains(fn, "setup")
+  return endswith(fn, ".jl") && startswith(basename(fn), "test_") && !contains(fn, "setup")
 end
 "match files of the form `*.jl`, but exclude `*_notest.jl` and `*setup*.jl`"
 function isexamplefile(fn)
-  endswith(fn, ".jl") && !endswith(fn, "_notest.jl") && !contains(fn, "setup")
+  return endswith(fn, ".jl") && !endswith(fn, "_notest.jl") && !contains(fn, "setup")
 end
 
 @time begin