Merge pull request #12 from metabase/process-docs-linting-and-test-branches

Author: escherize

.github/workflows/process_docs_changes.yml

@@ -55,21 +55,21 @@ jobs:
           # ./script/docs master --set-version master
           ./script/docs "$MAIN_REPO_BRANCH" --set-version "$MAIN_REPO_BRANCH"
 
-      # - name: "Lint the markdown"
-      #   run: |
-      #     yarn lint-markdown
+      - name: "Lint the markdown"
+        run: |
+          yarn lint-markdown
 
-      # - name: "Lint the styles"
-      #   run: |
-      #     yarn lint-styles
+      - name: "Lint the styles"
+        run: |
+          yarn lint-styles
 
-      # - name: "Lint the scripts"
-      #   run: |
-      #     yarn lint-scripts
+      - name: "Lint the scripts"
+        run: |
+          yarn lint-scripts
 
-      # - name: "Validate the links"
-      #   run: |
-      #     yarn lint-links
+      - name: "Validate the links"
+        run: |
+          yarn lint-links
 
       - name: Build the Jekyll Site
         env:
@@ -81,12 +81,12 @@ jobs:
           # Build the jekyll site:
           JEKYLL_ENV=development bundle exec jekyll build --baseurl '' --config _config.docs.yml --trace
 
-      # - name: "Check all of the links"
-      #   run: |
-      #     script/links || true
-      #     echo 'checking reported links...'
-      #     echo "htmlproofer spit out a report of length: $(cat htmlproofer.out | wc -l)"
-      #     bb script/analyze_links.clj --htmlproofer-output htmlproofer.out
+      - name: "Check all of the links"
+        run: |
+          script/links || true
+          echo 'checking reported links...'
+          echo "htmlproofer spit out a report of length: $(cat htmlproofer.out | wc -l)"
+          bb script/analyze_links.clj --htmlproofer-output htmlproofer.out
 
       - name: Authenticate Git with GITHUB_TOKEN
         run: |

script/check_incoming_branchname.clj

@@ -13,7 +13,8 @@
 (defn categorize-branchname [branchname]
   (cond
     (= branchname "master") [:master]
-    (re-matches release-regex branchname) [:release (extract-release-num branchname)]))
+    (re-matches release-regex branchname) [:release (extract-release-num branchname)]
+    (str/starts-with? branchname "docs-workflow-test-") [:test branchname]))
 
 (defn -main
   "This is called from the `process_docs_changes.yml` workflow. It checks that
@@ -26,7 +27,8 @@
         [category release-num] (categorize-branchname branchname)]
     (case category
       :master (println "Master branch detected.")
-      :release (println "Release branch detected:" release-num)
+      :release (println "Release branch detected. Release number: " release-num)
+      :test (println "Test branch detected. Branchname: " branchname)
       (do (println "Unpublishable branchname: " branchname)
           (System/exit 1)))))