Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

Feat/introduce web client #307

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 16 commits into
base: main
Choose a base branch
Loading
from
Open

Feat/introduce web client #307

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
1fc0d78
Split grpc connection out into node and web impls
tsmith023 Jun 10, 2025
589a0ab
Merge branch 'feat/web-client' of https://github.com/weaviate/typescr…
tsmith023 Jun 17, 2025
f350e2d
Commit changes:
tsmith023 Jun 20, 2025
3cfe9cd
Correct typos in CI
tsmith023 Jun 20, 2025
f088194
Build core in checks step
tsmith023 Jun 20, 2025
487e301
Add `pnpm` to steps, improve `npm run ci` cmd
tsmith023 Jun 20, 2025
69075cb
Change order of pnpm and node installs in CI
tsmith023 Jun 20, 2025
920e09b
Remove ellipsis from intsall cmds
tsmith023 Jun 20, 2025
fa6fe74
Remove `install` script from `package.json` root
tsmith023 Jun 20, 2025
8950c1c
Fix `build:all` typo in CI
tsmith023 Jun 20, 2025
7109d89
Split `npm run ci` script into step-specific cmds
tsmith023 Jun 20, 2025
9ecdd69
Fix `ci:test` script
tsmith023 Jun 20, 2025
6362c4c
Add missing dev deps to @weaviate/test
tsmith023 Jun 20, 2025
b1bab4b
Add missing curveball deps to @weaviate/test
tsmith023 Jun 20, 2025
350a694
Remove `it.only` from config test
tsmith023 Jun 20, 2025
49a0ce1
Merge branch 'main' of https://github.com/weaviate/typescript-client …
tsmith023 Jun 20, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
50 changes: 33 additions & 17 deletions 50 .github/workflows/main.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,15 +25,20 @@ jobs:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3
- uses: actions/setup-node@v3
- uses: pnpm/action-setup@v4
with:
node-version: '22.x'
version: 10
- uses: actions/setup-node@v4
with:
node-version: 22
cache: 'pnpm'
- name: "Run checks"
run: |
npm ci
npm run ci:checks
npm run lint
npm run format:check
npm run docs
npm run build

tests-without-auth:
needs: checks
Expand All @@ -54,9 +59,13 @@ jobs:
]
steps:
- uses: actions/checkout@v3
- uses: actions/setup-node@v3
- uses: pnpm/action-setup@v4
with:
version: 10
- uses: actions/setup-node@v4
with:
node-version: ${{ matrix.versions.node }}
node-version: 22
cache: 'pnpm'
- name: Login to Docker Hub
if: ${{ !github.event.pull_request.head.repo.fork && github.triggering_actor != 'dependabot[bot]' }}
uses: docker/login-action@v3
Expand All @@ -65,12 +74,10 @@ jobs:
password: ${{secrets.DOCKER_PASSWORD}}
- name: "Install dependencies"
run: |
npm ci
npm run ci:test
ci/run_dependencies.sh ${{ matrix.versions.weaviate }}
- name: "Run tests without authentication tests"
run: WEAVIATE_VERSION=${{ matrix.versions.weaviate }} npm test
- name: "Transpile the package"
run: npm run build
run: WEAVIATE_VERSION=${{ matrix.versions.weaviate }} npm run test
- name: "Stop Weaviate"
run: ci/stop_dependencies.sh ${{ matrix.versions.weaviate }}

Expand All @@ -85,9 +92,13 @@ jobs:
]
steps:
- uses: actions/checkout@v3
- uses: actions/setup-node@v3
- uses: pnpm/action-setup@v4
with:
node-version: ${{ matrix.versions.node }}
version: 10
- uses: actions/setup-node@v4
with:
node-version: 22
cache: 'pnpm'
- name: Login to Docker Hub
if: ${{ !github.event.pull_request.head.repo.fork && github.triggering_actor != 'dependabot[bot]' }}
uses: docker/login-action@v3
Expand All @@ -96,7 +107,7 @@ jobs:
password: ${{secrets.DOCKER_PASSWORD}}
- name: "Install dependencies"
run: |
npm ci
npm run ci:test
ci/run_dependencies.sh ${{ matrix.versions.weaviate }}
- name: "Run tests with authentication tests"
if: ${{ !github.event.pull_request.head.repo.fork }}
Expand All @@ -105,7 +116,7 @@ jobs:
WCS_DUMMY_CI_PW: ${{ secrets.WCS_DUMMY_CI_PW }}
AZURE_CLIENT_SECRET: ${{ secrets.AZURE_CLIENT_SECRET }}
OKTA_CLIENT_SECRET: ${{ secrets.OKTA_CLIENT_SECRET }}
run: WEAVIATE_VERSION=${{ matrix.versions.weaviate }} npm test
run: WEAVIATE_VERSION=${{ matrix.versions.weaviate }} npm run test
- name: "Stop Weaviate"
run: ci/stop_dependencies.sh ${{ matrix.versions.weaviate }}

Expand All @@ -120,12 +131,17 @@ jobs:
steps:
- uses: actions/checkout@v3
# Setup .npmrc file to publish to npm
- uses: actions/setup-node@v3
- uses: pnpm/action-setup@v4
with:
version: 10
- uses: actions/setup-node@v4
with:
node-version: '22.x'
node-version: 22
cache: 'pnpm'
registry-url: 'https://registry.npmjs.org'
- run: npm ci
- run: npm run build
- run: npm run ci:publish
- run: npm run build:node
- run: npm run build:web
- run: npm publish
env:
NODE_AUTH_TOKEN: ${{ secrets.NPM_AUTOMATION_TOKEN }}
Expand Down
2 changes: 1 addition & 1 deletion 2 .husky/pre-commit
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
#!/bin/sh
. "$(dirname "$0")/_/husky.sh"

npx lint-staged
npm run lint:staged
6 changes: 3 additions & 3 deletions 6 .prettierignore
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
src/proto/**/*.ts
dist/**/*
docs/**/*.js
**/proto/**/*.ts
**/dist/**/*
**/docs/**/*.js
Loading
Loading
Morty Proxy This is a proxified and sanitized view of the page, visit original site.