ci: update tests workflow to use centralised reusable workflow and au…

…to-suggest format changes (#159)

* ci: update tests workflow to use centralised reusable workflow

* ci: remove `FormatCheck` workflow

Remove the `FormatCheck` workflow, since code in the repository should
always be compliant with the SciML style, thus eliminating the need for
automated formatting PRs.

* ci(format-check): automatically comment formatting suggestions on PRs

* chore: rename to BACKEND_GROUP

* revert: rename to BACKEND_GROUP

* fix: fallback to GROUP

* fix: fallback to GROUP

* fix: typo

---------

Co-authored-by: Avik Pal <avikpal@mit.edu>

.github/workflows/FormatCheck.yml

@@ -10,4 +10,4 @@ on:
 jobs:
   format-check:
     name: "Format Check"
-    uses: "SciML/.github/.github/workflows/format-check.yml@v1"
+    uses: "SciML/.github/.github/workflows/format-suggestions-on-pr.yml@v1"

.github/workflows/Tests.yml

@@ -0,0 +1,25 @@
+name: "Tests"
+
+on:
+  pull_request:
+    branches:
+      - main
+    paths-ignore:
+      - 'docs/**'
+  push:
+    branches:
+      - main
+    paths-ignore:
+      - 'docs/**'
+
+concurrency:
+  group: ${{ github.workflow }}-${{ github.ref }}
+  cancel-in-progress: ${{ github.ref_name != github.event.repository.default_branch || github.ref != 'refs/tags/v*' }}
+
+jobs:
+  tests:
+    name: "Tests"
+    uses: "SciML/.github/.github/workflows/tests.yml@v1"
+    with:
+      group: "CPU"
+    secrets: "inherit"

test/runtests.jl

@@ -2,7 +2,7 @@ using ReTestItems, Pkg, InteractiveUtils, Hwloc
 
 @info sprint(versioninfo)
 
-const BACKEND_GROUP = lowercase(get(ENV, "BACKEND_GROUP", "all"))
+const BACKEND_GROUP = lowercase(get(ENV, "BACKEND_GROUP", get(ENV, "GROUP", "all")))
 const EXTRA_PKGS = String[]
 
 (BACKEND_GROUP == "all" || BACKEND_GROUP == "cuda") && push!(EXTRA_PKGS, "LuxCUDA")

test/shared_testsetup.jl

@@ -6,7 +6,7 @@ using MLDataDevices, GPUArraysCore
 
 LuxTestUtils.jet_target_modules!(["DeepEquilibriumNetworks", "Lux", "LuxLib"])
 
-const BACKEND_GROUP = lowercase(get(ENV, "BACKEND_GROUP", "all"))
+const BACKEND_GROUP = lowercase(get(ENV, "BACKEND_GROUP", get(ENV, "GROUP", "all")))
 
 if BACKEND_GROUP == "all" || BACKEND_GROUP == "cuda"
     using LuxCUDA