Skip to content

Commit

Permalink
Merge pull request #13 from getappmap/fix/no-check-step
Browse files Browse the repository at this point in the history
ci: Remove preflight check step
  • Loading branch information
kgilpin authored May 18, 2023
2 parents 8556a45 + cfe85e7 commit 6ee786d
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 9 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/appmap-archive.yml
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ jobs:
ln -s /usr/local/bin/appmap /tmp/appmap
- name: Build AppMaps
run: yarn appmap
run: yarn test

- name: Archive AppMaps
uses: getappmap/[email protected]
Expand Down
7 changes: 1 addition & 6 deletions .github/workflows/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ jobs:
run: yarn build

- name: Test
run: yarn appmap || true
run: yarn test

- name: Preflight
id: preflight
Expand All @@ -41,11 +41,6 @@ jobs:
head-revision: ${{ github.event.pull_request.head.sha }}
github-token: ${{ secrets.GITHUB_TOKEN }}

- name: Check test status
uses: ./check
with:
directory: ${{ steps.preflight.outputs.report-dir }}

- name: Build GitHub action
run: yarn package

Expand Down
3 changes: 1 addition & 2 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -9,8 +9,7 @@
"watch": "tsc --watch",
"package": "ncc build -o dist/preflight --source-map src/preflight.ts && ncc build -o dist/check --source-map src/check.ts",
"build": "yarn compile && yarn package",
"test": "jest --runInBand",
"appmap": "appmap-agent-js -- jest --runInBand --no-cache",
"test": "appmap-agent-js -- jest --runInBand --no-cache",
"clean": "rm -rf build dist tmp",
"format": "prettier --write '**/*.ts'"
},
Expand Down

0 comments on commit 6ee786d

Please sign in to comment.