diff --git a/.github/workflows/test-all-packages.yml b/.github/workflows/test-all-packages.yml index 6b1d17313a4..e17b56ee0ee 100644 --- a/.github/workflows/test-all-packages.yml +++ b/.github/workflows/test-all-packages.yml @@ -66,12 +66,10 @@ jobs: with: node-version: '18.x' - # first job also does repo-level linting - name: lint repo format run: yarn lint:format - # eslint - - name: yarn lint primary - run: ./scripts/lint-with-types.sh primary + # eslint and tsc + - run: yarn lint:packages # build the API docs to verify it works - name: build API docs run: yarn docs @@ -79,18 +77,11 @@ jobs: - name: build API docs in markdown run: yarn docs:markdown-for-agoric-documentation-repo + # UNTIL repo settings are updated to not require it lint-rest: - timeout-minutes: 15 - needs: build runs-on: ubuntu-latest steps: - - uses: actions/checkout@v4 - - uses: ./.github/actions/restore-node - with: - node-version: '18.x' - - - name: yarn lint rest - run: ./scripts/lint-with-types.sh rest + - run: true ################## # Fast-running tests run as a group: diff --git a/scripts/lint-with-types.sh b/scripts/lint-with-types.sh deleted file mode 100755 index a2821b626b0..00000000000 --- a/scripts/lint-with-types.sh +++ /dev/null @@ -1,22 +0,0 @@ -#!/bin/sh - -# TODO consolidate back into one job https://github.com/Agoric/agoric-sdk/pull/8061 - -# argument used by CI to split this across two jobs -SCOPE=$1 - -# taking roughly half the time to eslint all packages -PRIMARY_PACKAGES="@agoric/{cosmos,ertp,governance,inter-protocol,swing-store,swingset-vat,vats,wallet,zoe}" - -case $SCOPE in - primary) - yarn lerna run --scope=$PRIMARY_PACKAGES --no-bail lint - ;; - rest) - yarn lerna run --ignore=$PRIMARY_PACKAGES --no-bail lint - ;; - *) - echo "The regular lint command now lints with types. Just use that." - exit 0 - ;; -esac