From: aescling <aescling+gitlab@cat.family>
Date: Mon, 29 Aug 2022 06:56:35 +0000 (-0400)
Subject: CI: Support larger diff artifacts
X-Git-Url: https://git.xn--scling-oua.cat.family/?a=commitdiff_plain;h=0f966f11bbcbec357c160b6cc62c96cd6c083c7c;p=mastodon.git

CI: Support larger diff artifacts

(Much too large) MRs can theoretically exceed the artifact size limit
for dotenv files; this ought to be more flexible for us.

A previous attempt just invoked deno as needed per stage, but this
involved installing deno as needed in non-deno images, and would be
ineffecient if we ever need the other diffs in any stage.
---

diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index ae36052c3..bee338a39 100644
--- a/.gitlab-ci.yml
+++ b/.gitlab-ci.yml
@@ -82,15 +82,20 @@ diff:
         { all: [], rb: [], js: [] },
       );
       Deno.writeTextFileSync(
-        'build.env',
-        'CHANGED_FILES=' + all.join(' ') + String.fromCodePoint(0x0A) +
-          'CHANGED_RUBY=' + rb.join(' ') + String.fromCodePoint(0x0A) +
-          'CHANGED_JS=' + js.join(' ') + String.fromCodePoint(0x0A),
+        'diff-all.txt', all.join(' ') + String.fromCodePoint(0x0A)
+      );
+      Deno.writeTextFileSync(
+        'diff-rb.txt', rb.join(' ') + String.fromCodePoint(0x0A)
+      );
+      Deno.writeTextFileSync(
+        'diff-js.txt', js.join(' ') + String.fromCodePoint(0x0A)
       );"
-    - cat build.env
+    - 'cat diff-*.txt'
   artifacts:
-    reports:
-      dotenv: build.env
+    paths:
+      - diff-all.txt
+      - diff-rb.txt
+      - diff-js.txt
 
 build:
   stage: build
@@ -106,7 +111,7 @@ build:
 lint:
   stage: test
   script:
-    - 'if [ ! -z "$CHANGED_RUBY" ]; then bundle exec standardrb $CHANGED_RUBY; fi'
+    - 'if [ "$(exec wc -c <diff-rb.txt)" -gt 1 ]; then bundle exec standardrb $(exec cat diff-rb.txt); fi' # newline is always present
   allow_failure: true
 
 test: