Skip to content

Commit

Permalink
remove extra integration test cleanup, remove publishing from merge w…
Browse files Browse the repository at this point in the history
…orkflow for now
  • Loading branch information
austin-denoble committed Jan 13, 2024
1 parent d3d4e50 commit d022826
Show file tree
Hide file tree
Showing 2 changed files with 15 additions and 27 deletions.
19 changes: 15 additions & 4 deletions .github/workflows/merge.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,18 @@ on:
workflow_dispatch: {}

jobs:
build-and-publish-docs:
name: Build and publish docs to sdk-docs
uses: ./.github/workflows/build-and-publish-docs.yml
secrets: inherit
build-docs:
name: Build sdk-docs
runs-on: ubuntu-latest
steps:
- name: Checkout
uses: actions/checkout@v4
- name: Setup
uses: ./.github/actions/setup
- name: Build docs
uses: ./github/actions/build-docs

# build-and-publish-docs:
# name: Build and publish docs to sdk-docs
# uses: ./.github/workflows/build-and-publish-docs.yml
# secrets: inherit
23 changes: 0 additions & 23 deletions .github/workflows/testing.yml
Original file line number Diff line number Diff line change
Expand Up @@ -68,29 +68,6 @@ jobs:
PINECONE_CONTROLLER_HOST: 'https://api-staging.pinecone.io'
run: ${{ matrix.config.runner }} run test:integration:${{ matrix.config.jest_env }}

integration-test-cleanup:
name: Clean up integration tests
if: always()
needs: [integration-tests]
runs-on: ubuntu-latest
steps:
- name: Checkout
uses: actions/checkout@v4
- name: Setup Node
uses: actions/setup-node@v3
with:
node-version: 18.x
registry-url: 'https://registry.npmjs.org'
- name: Install npm packages
run: |
npm install --ignore-scripts
shell: bash
- name: Run integration cleanup command
env:
CI: true
PINECONE_API_KEY: ${{ secrets.PINECONE_API_KEY }}
run: npm run test:integration:cleanup

typescript-compilation-tests:
name: TS compile
runs-on: ubuntu-latest
Expand Down

0 comments on commit d022826

Please sign in to comment.