test: fix Aqua workflow (#59)

Author: mortenpi

.github/workflows/CI.yml

@@ -73,16 +73,14 @@ jobs:
       - uses: julia-actions/setup-julia@v1
         with:
           version: "1"
-      - run: |
+      - name: "Aqua.test_all(JuliaHub)"
+        shell: julia --color=yes {0}
+        run: |
           import Pkg
           Pkg.activate(temp=true)
-          Pkg.add(name="Aqua", version="0.6")
+          Pkg.add(name="Aqua", version="0.8")
           Pkg.develop(path=".")
-          import Aqua, JuliaHub
-          Aqua.test_all(JuliaHub)
-        shell: julia --color=yes {0}
-        name: "Aqua.test_all(JuliaHub)"
-        continue-on-error: true
+          include(joinpath(pwd(), "test", "aqua.jl"))
 
   jet:
     name: JET

test/aqua.jl

@@ -0,0 +1,20 @@
+import Aqua
+import JuliaHub
+using Test
+
+@testset "Aqua" begin
+    Aqua.test_all(
+        JuliaHub;
+        # These ambiguities are introduced by InlineStrings
+        # https://github.com/JuliaStrings/InlineStrings.jl/issues/71
+        ambiguities=(;
+            exclude=[
+                Base.Sort.defalg
+            ],
+        ),
+        # Aqua detects missing standard library compat entries, but setting these
+        # is problematic if we want to support 1.6
+        # https://discourse.julialang.org/t/psa-compat-requirements-in-the-general-registry-are-changing/104958
+        deps_compat=(; broken=true, check_extras=false),
+    )
+end