Speed up github actions for PRs

.github/workflows/reusable-test.yml

@@ -163,7 +163,7 @@ jobs:
       fail-fast: false
       matrix:
         node: ${{ fromJSON(inputs.node_version) }}
-        browser: ["edge"]
+        browser: ["msedge"]
 
     runs-on: windows-latest
     steps:

.github/workflows/shared-build.yml

@@ -0,0 +1,37 @@
+name: 🛠️ Build
+
+on:
+  workflow_call:
+
+env:
+  CI: true
+  CYPRESS_INSTALL_BINARY: 0
+
+jobs:
+  build:
+    name: ⚙️ Build
+    runs-on: ubuntu-latest
+    steps:
+      - name: 🛑 Cancel Previous Runs
+        uses: styfle/[email protected]
+
+      - name: ⬇️ Checkout repo
+        uses: actions/checkout@v3
+
+      - name: ⎔ Setup node
+        uses: actions/setup-node@v3
+        with:
+          node-version-file: ".nvmrc"
+          cache: "yarn"
+
+      - name: Disable GitHub Actions Annotations
+        run: |
+          echo "::remove-matcher owner=tsc::"
+          echo "::remove-matcher owner=eslint-compact::"
+          echo "::remove-matcher owner=eslint-stylish::"
+
+      - name: 📥 Install deps
+        run: yarn --frozen-lockfile
+
+      - name: 🏗 Build
+        run: yarn build

.github/workflows/shared-test-integration.yml

@@ -0,0 +1,62 @@
+name: 🧪 Test (Integration)
+
+on:
+  workflow_call:
+    inputs:
+      os:
+        required: true
+        type: string
+      node_version:
+        required: true
+        # this is limited to string | boolean | number (https://github.community/t/can-action-inputs-be-arrays/16457)
+        # but we want to pass an array (node_version: "[18, 20]"),
+        # so we'll need to manually stringify it for now
+        type: string
+      browser:
+        required: true
+        # this is limited to string | boolean | number (https://github.community/t/can-action-inputs-be-arrays/16457)
+        # but we want to pass an array (browser: "['chromium', 'firefox']"),
+        # so we'll need to manually stringify it for now
+        type: string
+
+env:
+  CI: true
+  CYPRESS_INSTALL_BINARY: 0
+
+jobs:
+  integration:
+    name: '👀 Test (${{ inputs.os }}/node@${{ matrix.node }}/${{ matrix.browser }})'
+    strategy:
+      fail-fast: false
+      matrix:
+        node: ${{ fromJSON(inputs.node_version) }}
+        browser: ${{ fromJSON(inputs.browser) }}
+
+    runs-on: ${{ inputs.os }}
+    steps:
+      - name: 🛑 Cancel Previous Runs
+        uses: styfle/[email protected]
+
+      - name: ⬇️ Checkout repo
+        uses: actions/checkout@v3
+
+      - name: ⎔ Setup node ${{ matrix.node }}
+        uses: actions/setup-node@v3
+        with:
+          node-version: ${{ matrix.node }}
+          cache: "yarn"
+
+      - name: Disable GitHub Actions Annotations
+        run: |
+          echo "::remove-matcher owner=tsc::"
+          echo "::remove-matcher owner=eslint-compact::"
+          echo "::remove-matcher owner=eslint-stylish::"
+
+      - name: 📥 Install deps
+        run: yarn --frozen-lockfile
+
+      - name: 📥 Install Playwright
+        run: npx playwright install --with-deps ${{ matrix.browser }}
+
+      - name: 👀 Run Integration Tests ${{ matrix.browser }}
+        run: "yarn test:integration --project=${{ matrix.browser }}"

.github/workflows/shared-test-unit.yml

@@ -0,0 +1,55 @@
+name: 🧪 Test (Unit)
+
+on:
+  workflow_call:
+    inputs:
+      os:
+        required: true
+        type: string
+      node_version:
+        required: true
+        # this is limited to string | boolean | number (https://github.community/t/can-action-inputs-be-arrays/16457)
+        # but we want to pass an array (node_version: "[18, 20]"),
+        # so we'll need to manually stringify it for now
+        type: string
+
+env:
+  CI: true
+  CYPRESS_INSTALL_BINARY: 0
+
+jobs:
+  test:
+    name: '🧪 Test (${{ inputs.os }}/node@${{ matrix.node }})'
+    strategy:
+      fail-fast: false
+      matrix:
+        node: ${{ fromJSON(inputs.node_version) }}
+    runs-on: ${{ inputs.os }}
+    steps:
+      - name: 🛑 Cancel Previous Runs
+        uses: styfle/[email protected]
+
+      - name: ⬇️ Checkout repo
+        uses: actions/checkout@v3
+
+      - name: ⎔ Setup node ${{ matrix.node }}
+        uses: actions/setup-node@v3
+        with:
+          node-version: ${{ matrix.node }}
+          cache: "yarn"
+
+      - name: Disable GitHub Actions Annotations
+        run: |
+          echo "::remove-matcher owner=tsc::"
+          echo "::remove-matcher owner=eslint-compact::"
+          echo "::remove-matcher owner=eslint-stylish::"
+
+      - name: 📥 Install deps
+        run: yarn --frozen-lockfile
+
+      # It's faster to use the built `cli.js` in tests if its available and up-to-date
+      - name: 🏗 Build
+        run: yarn build
+
+      - name: 🧪 Run Primary Tests
+        run: "yarn test:primary"

.github/workflows/test-full.yml

@@ -0,0 +1,50 @@
+name: Branch
+
+# main/dev branches will get the full run across all OS/browsers
+
+on:
+  push:
+    branches:
+      - main
+      - dev
+    paths-ignore:
+      - "docs/**"
+      - "scripts/**"
+      - "contributors.yml"
+      - "**/*.md"
+
+jobs:
+  build:
+    if: github.repository == 'remix-run/remix'
+    uses: ./.github/workflows/shared-build.yml
+
+  unit:
+    if: github.repository == 'remix-run/remix'
+    uses: ./.github/workflows/shared-test-unit.yml
+    with:
+      os: '["ubuntu-latest", "windows-latest"]'
+      node_version: '["latest"]'
+
+  ubuntu:
+    if: github.repository == 'remix-run/remix'
+    uses: ./.github/workflows/shared-test-integration.yml
+    with:
+      os: '["ubuntu-latest"]'
+      node_version: '["latest"]'
+      browser: '["chromium", "firefox"]'
+
+  windows:
+    if: github.repository == 'remix-run/remix'
+    uses: ./.github/workflows/shared-test-integration.yml
+    with:
+      os: '["windows-latest"]'
+      node_version: '["latest"]'
+      browser: '["msedge"]'
+
+  macos:
+    if: github.repository == 'remix-run/remix'
+    uses: ./.github/workflows/shared-test-integration.yml
+    with:
+      os: '["macos-latest"]'
+      node_version: '["latest"]'
+      browser: '["webkit"]'

.github/workflows/test-pr-ubuntu.yml

@@ -0,0 +1,31 @@
+name: PR (Base)
+
+# All PRs touching code will run tests on ubuntu/node/chromium
+
+on:
+  pull_request:
+    paths-ignore:
+      - "docs/**"
+      - "scripts/**"
+      - "contributors.yml"
+      - "**/*.md"
+
+jobs:
+  build:
+    if: github.repository == 'remix-run/remix'
+    uses: ./.github/workflows/shared-build.yml
+
+  unit-ubuntu:
+    if: github.repository == 'remix-run/remix'
+    uses: ./.github/workflows/shared-test-unit.yml
+    with:
+      os: "ubuntu-latest"
+      node_version: '["latest"]'
+
+  integration-chromium:
+    if: github.repository == 'remix-run/remix'
+    uses: ./.github/workflows/shared-test-integration.yml
+    with:
+      os: "ubuntu-latest"
+      node_version: '["latest"]'
+      browser: '["chromium"]'

.github/workflows/test-pr-windows-macos.yml

@@ -0,0 +1,42 @@
+name: PR (Full)
+
+# PRs touching create-remix/remix-dev will also run on Windows and OSX
+
+on:
+  pull_request:
+    paths:
+      - "packages/create-remix/**"
+      - "packages/remix-dev/**"
+      - "!**/*.md"
+
+jobs:
+  unit-windows:
+    if: github.repository == 'remix-run/remix'
+    uses: ./.github/workflows/shared-test-unit.yml
+    with:
+      os: "windows-latest"
+      node_version: '["latest"]'
+
+  integration-firefox:
+    if: github.repository == 'remix-run/remix'
+    uses: ./.github/workflows/shared-test-integration.yml
+    with:
+      os: "ubuntu-latest"
+      node_version: '["latest"]'
+      browser: '["firefox"]'
+
+  integration-msedge:
+    if: github.repository == 'remix-run/remix'
+    uses: ./.github/workflows/shared-test-integration.yml
+    with:
+      os: "windows-latest"
+      node_version: '["latest"]'
+      browser: '["msedge"]'
+
+  integration-webkit:
+    if: github.repository == 'remix-run/remix'
+    uses: ./.github/workflows/shared-test-integration.yml
+    with:
+      os: "macos-latest"
+      node_version: '["latest"]'
+      browser: '["webkit"]'

integration/hmr-log-test.ts

@@ -510,7 +510,7 @@ whatsup
     // let expectedErrorCount = 0;
     let expectDestructureTypeError = expectConsoleError((error) => {
       let expectedMessage = new Set([
-        // chrome, edge
+        // chrome, msedge
         "Cannot destructure property 'hello' of 'useLoaderData(...)' as it is null.",
         // firefox
         "(intermediate value)() is null",

integration/hmr-test.ts

@@ -590,7 +590,7 @@ whatsup
     // let expectedErrorCount = 0;
     let expectDestructureTypeError = expectConsoleError((error) => {
       let expectedMessage = new Set([
-        // chrome, edge
+        // chrome, msedge
         "Cannot destructure property 'hello' of 'useLoaderData(...)' as it is null.",
         // firefox
         "(intermediate value)() is null",

integration/playwright.config.ts

@@ -26,7 +26,7 @@ const config: PlaywrightTestConfig = {
       use: devices["Desktop Safari"],
     },
     {
-      name: "edge",
+      name: "msedge",
       use: devices["Desktop Edge"],
     },
     {

packages/create-remix/index.ts

@@ -34,6 +34,8 @@ import { renderLoadingIndicator } from "./loading-indicator";
 import { copyTemplate, CopyTemplateError } from "./copy-template";
 import { getLatestRemixVersion } from "./remix-version";
 
+// TEST CHANGE TO TRIGGER WORKFLOWS
+
 async function createRemix(argv: string[]) {
   let ctx = await getContext(argv);
   if (ctx.help) {

packages/remix-react/components.tsx

@@ -65,6 +65,8 @@ function useDataRouterStateContext() {
   return context;
 }
 
+// TEST CHANGE TO TRIGGER WORKFLOWS
+
 ////////////////////////////////////////////////////////////////////////////////
 // RemixContext