-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
ci[patch]: fix unit-tests-integrations by implementing dynamic matrix strategy. #5768
Conversation
rameshkrishna
commented
Jun 15, 2024
•
edited
Loading
edited
- get-changed-files: Determines the files changed in the latest commit.
- prepare-matrix: Generates a matrix strategy for packages that have changed and need testing.
- unit-tests: Runs the unit tests for the identified packages on Node.js versions 18 and 20 incase of no changes in unit test jobs is skipped and whole workflow gets marked as success.
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
What issue is this fixing? @rameshkrishna |
Hey @bracesproul, This PR addresses the unit tests langchain integrations workflow failures caused by the changes introduced in PR ci[patch]: Only run CI testing if specific package touched 5733. Failures in this github workflow : https://github.com/langchain-ai/langchainjs/actions/workflows/unit-tests-integrations.yml Here is the error message when the above work flow is triggered Annotations1 error |
More details here #5812 |
Google firestore vectorstore
…ctorstore Revert "Google firestore vectorstore"
CC @bracesproul is this still needed? |