-
Notifications
You must be signed in to change notification settings - Fork 0
Query agent #1
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.
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
Query agent #1
Conversation
This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Orca Security Scan Summary
Status | Check | Issues by priority | |
---|---|---|---|
![]() |
Secrets | ![]() ![]() ![]() ![]() |
View in Orca |
Everything looks good @augustas1 ! Do you have any tests planned for the repo? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I couple files are missing, LICENSE, README etc, I could help with some of these as I work on examples if you'd like.
"keywords": [ | ||
"weaviate" | ||
], | ||
"author": "Weaviate", | ||
"license": "SEE LICENSE IN LICENSE" | ||
"license": "SEE LICENSE IN LICENSE", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the LICENSE file is missing @augustas1
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it's already on main
Yes, I'll add README, tests and CI later on. Didn't want to have a mega PR :). |
@augustas1 is there a rough timeline for when we could starting using the QueryAgent in TypeScript? 😍 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For the CI, feel free to copy what you want/need from the TS client's workflow file
@IchordeDionysos I think Query Agent should be there in 1-2 weeks |
Initial Query Agent pull request.
Depends on weaviate/typescript-client#292.
Has
weaviate-client
as a peer dependency and can be consumed like this:# once published npm install weaviate-client @weaviate/agents