Merge pull request #278 from CliMA/ck/aqua

Update Aqua

test/Project.toml

@@ -24,7 +24,7 @@ StaticArrays = "90137ffa-7385-5640-81b9-e52037218182"
 Test = "8dfed614-e22c-5e08-85e1-65c5234f0b40"
 
 [compat]
-Aqua = "0.6"
+Aqua = "0.8"
 CUDA = "3, 4, 5"
 ClimaComms = "0.4, 0.5, 0.6"
 ClimaCore = "0.10, 0.11, 0.12"

test/aqua.jl

@@ -3,12 +3,6 @@ using ClimaTimeSteppers
 using Aqua
 
 @testset "Aqua tests (performance)" begin
-    # This tests that we don't accidentally run into
-    # https://github.com/JuliaLang/julia/issues/29393
-    # Aqua.test_unbound_args(ClimaTimeSteppers)
-    ua = Aqua.detect_unbound_args_recursively(ClimaTimeSteppers)
-    @test length(ua) == 0
-
     # See: https://github.com/SciML/OrdinaryDiffEq.jl/issues/1750
     # Test that we're not introducing method ambiguities across deps
     ambs = Aqua.detect_ambiguities(ClimaTimeSteppers; recursive = true)
@@ -23,13 +17,8 @@ using Aqua
     @test length(ambs) == 0
 end
 
-@testset "Aqua tests (additional)" begin
-    Aqua.test_undefined_exports(ClimaTimeSteppers)
-    Aqua.test_stale_deps(ClimaTimeSteppers)
-    Aqua.test_deps_compat(ClimaTimeSteppers)
-    Aqua.test_project_extras(ClimaTimeSteppers)
-    Aqua.test_project_toml_formatting(ClimaTimeSteppers)
-    Aqua.test_piracy(ClimaTimeSteppers)
+@testset "Aqua tests (remaining)" begin
+    Aqua.test_all(ClimaTimeSteppers; ambiguities=false)
 end
 
 nothing