-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[BLD]: JS client 1.9.1-alpha release #2150
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Reviewer ChecklistPlease leverage this checklist to ensure your code review is thorough before approving Testing, Bugs, Errors, Logs, Documentation
System Compatibility
Quality
|
5b0252b
to
0e8f75a
Compare
Hi Trayan, I'm OOO most this week but I should be able to take a look Friday. |
@@ -12,4 +12,10 @@ services: | |||
- ALLOW_RESET=True | |||
- IS_PERSISTENT=True | |||
ports: | |||
- 8000:8000 | |||
- "${CHROMA_PORT:-8000}:8000" |
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.
@codetheweb I python tests don't like the default here. Lemme check.
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.
actually it's the test_net.
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.
looks like it was resolved?
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.
weird failure in linting. Reran, it's all fine now.
574dcfa
to
0e8f75a
Compare
0e8f75a
to
8e382db
Compare
## Description of changes *Summarize the changes made by this PR.* - Improvements & Bug fixes - JS version bump to `1.9.1` - Made the error handling more robust for offline errors (works with older Nodejs 16) - Fixed an issue with test docker-compose file where the port was not being correctly mapped, thus making the release WF tests fail ## Test plan *How are these changes tested?* - [x] Tests pass locally with `pytest` for python, `yarn test` for js, `cargo test` for rust ## Documentation Changes N/A
Description of changes
Summarize the changes made by this PR.
1.9.1
Test plan
How are these changes tested?
pytest
for python,yarn test
for js,cargo test
for rustDocumentation Changes
N/A