From eb55af1b1f2ec51635a48d15c8e22b3b3f2d61cf Mon Sep 17 00:00:00 2001
From: David Taylor <david@taylorhq.com>
Date: Wed, 22 Apr 2026 15:56:23 +0100
Subject: [PATCH] FEATURE: Allow opting out of linting/annotations

This is particularly useful if testing a plugin against an array of core versions. You only want linting/annotations for the main one
---
 .github/workflows/discourse-plugin.yml | 15 ++++++++++++++-
 .github/workflows/discourse-theme.yml  |  6 ++++++
 2 files changed, 20 insertions(+), 1 deletion(-)

diff --git a/.github/workflows/discourse-plugin.yml b/.github/workflows/discourse-plugin.yml
index 09abfb8..0efe627 100644
--- a/.github/workflows/discourse-plugin.yml
+++ b/.github/workflows/discourse-plugin.yml
@@ -20,6 +20,16 @@ on:
       container:
         type: string
         required: false
+      skip_linting:
+        description: "Skip the linting job."
+        type: boolean
+        required: false
+        default: false
+      skip_annotations:
+        description: "Skip the annotation checks in the test matrix."
+        type: boolean
+        required: false
+        default: false
     secrets:
       ssh_private_key:
         description: "Optional SSH private key to be used when cloning additional plugin repos"
@@ -38,6 +48,7 @@ defaults:
 
 jobs:
   linting:
+    if: ${{ !inputs.skip_linting }}
     runs-on: ${{ inputs.runs_on }}
     container: ${{ inputs.container || 'discourse/discourse_test:slim' }}
 
@@ -167,6 +178,8 @@ jobs:
         id: check_tests
         shell: ruby {0}
         working-directory: tmp/plugin
+        env:
+          SKIP_ANNOTATIONS: ${{ inputs.skip_annotations }}
         run: |
           require 'json'
 
@@ -175,7 +188,7 @@ jobs:
           matrix << 'frontend' if Dir.glob("test/javascripts/**/*.{js,gjs}").any?
           matrix << 'backend'
           matrix << 'system' if Dir.glob("spec/system/**/*.rb").any?
-          matrix << 'annotations' if Dir.glob("app/models/**/*.rb").reject { _1.match?("concerns") }.any?
+          matrix << 'annotations' if ENV["SKIP_ANNOTATIONS"] != "true" && Dir.glob("app/models/**/*.rb").reject { _1.match?("concerns") }.any?
 
           puts "Running jobs: #{matrix.inspect}"
 
diff --git a/.github/workflows/discourse-theme.yml b/.github/workflows/discourse-theme.yml
index b3e8d8d..67284d4 100644
--- a/.github/workflows/discourse-theme.yml
+++ b/.github/workflows/discourse-theme.yml
@@ -17,6 +17,11 @@ on:
       container:
         type: string
         required: false
+      skip_linting:
+        description: "Skip the linting job."
+        type: boolean
+        required: false
+        default: false
     secrets:
       ssh_private_key:
         description: "Optional SSH private key to be used when cloning additional plugin repos"
@@ -35,6 +40,7 @@ defaults:
 
 jobs:
   linting:
+    if: ${{ !inputs.skip_linting }}
     runs-on: ${{ inputs.runs_on }}
     container: ${{ inputs.container || 'discourse/discourse_test:slim' }}