Merge pull request #8 from getappmap/feat/refactor-workflows

feat: Refactor workflows

Author: kgilpin

.github/workflows/appmap-archive.yml

@@ -0,0 +1,29 @@
+on:
+  pull_request:
+    types:
+      - closed
+
+env:
+  APPMAP_TELEMETRY_DISABLED: 'true'
+
+jobs:
+  appmap_archive_main:
+    runs-on: ubuntu-latest
+    if: github.event.pull_request.merged == true
+    steps:
+      - name: Checkout step
+        uses: actions/checkout@v3
+      - name: Install dependencies
+        run: yarn install
+      - name: Install and configure AppMap tools
+        uses: getappmap/install-appmap-action@v1-pre.3
+        id: install-appmap
+        with:
+          verbose: true
+      - name: Build AppMaps
+        run: yarn appmap
+      - name: Archive AppMaps
+        uses: ./
+        id: archive-appmap
+        with:
+          verbose: true

.github/workflows/main.yml

@@ -10,24 +10,15 @@ jobs:
     steps:
       - name: Checkout
         uses: actions/checkout@v3
-      - name: Install and configure AppMap
-        uses: getappmap/install-appmap-action@v1-pre.2
-        id: install-appmap
       - name: Install dependency libraries
         run: yarn install
       - name: Build
         run: yarn build
       - name: Test
         run: yarn test
-      - name: Build GitHub action
+      - name: Package
         run: yarn package
-      - name: Make AppMaps
-        # appmap-agent-js@13.6.0 doesn't propagate the jest exit code, so the main Test step cannot be the AppMap step.
-        run: yarn appmap
-      - name: Create and store AppMaps archive
-        uses: ./
-        id: appmap-archive
-      - name: Commit changes
+      - name: Commit package changes, if any
         # If you forgot to package the distribution, you'll see a commit added to your PR
         uses: EndBug/add-and-commit@v7
         with: