fix: npm publish CI job lacked TZ environment which broke CI

ftrackhq · Feb 22, 2023 · 25475aa · 25475aa
1 parent 49f4a21
commit 25475aa
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 5 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -11,8 +11,6 @@ on:
 jobs:
   build:
     runs-on: ubuntu-latest
-    env:
-      TZ: Europe/Stockholm
     steps:
       - uses: actions/checkout@v3
       - name: Using Node.js from .nvmrc

diff --git a/.github/workflows/publish-npm.yml b/.github/workflows/publish-npm.yml
@@ -5,9 +5,6 @@ on:
     tags:
       - "v*"
 
-env:
-  TZ: Europe/Stockholm
-
 jobs:
   cd:
     name: Publish to NPM

diff --git a/vite.config.ts b/vite.config.ts
@@ -36,6 +36,7 @@ export default defineConfig({
   test: {
     environment: "jsdom",
     globals: true,
+    globalSetup: "./vitest.globalsetup.js",
     setupFiles: ["./vitest.setup.js"],
     deps: {
       fallbackCJS: true,

diff --git a/vitest.globalsetup.js b/vitest.globalsetup.js
@@ -0,0 +1,3 @@
+export const setup = () => {
+  process.env.TZ = "Europe/Stockholm";
+};
-Original file line number
+Diff line change
@@ Expand Up / @@ -5,9 +5,6 @@ on: @@
         tags:
           - "v*"
-    env:
-      TZ: Europe/Stockholm
     jobs:
       cd:
         name: Publish to NPM
@@ Expand Down @@