Skip to content

fix(deps): added fastembed back in - #254

Merged
rileyok-ons merged 1 commit into
mainfrom
253-re-add-fastembed
Nov 6, 2026
Merged

rileyok-ons merged 1 commit into
mainfrom
253-re-add-fastembed

Conversation

@rileyok-ons

Copy link
Copy Markdown
Contributor

📌 Pull Request Template

Please complete all sections

✨ Summary

This adds fastembed back in after it has been updated to support pillow 13.0.0, removing some 9.8 cves from its codebase.
Also provides updates for other deps that need patching

📜 Changes Introduced

  • Feature implementation (feat:) / bug fix (fix:) / refactoring (chore:) / documentation (docs:) / testing (test:)
  • Updates to tests and/or documentation
  • Terraform changes (if applicable)

✅ Checklist

Please confirm you've completed these checks before requesting a review.

  • Code passes linting with Ruff
  • Security checks pass using Bandit
  • API and Unit tests are written and pass using pytest
  • Terraform files (if applicable) follow best practices and have been validated (terraform fmt & terraform validate)
  • DocStrings follow Google-style and are added as per Pylint recommendations
  • Documentation has been updated if needed

🔍 How to Test

Run uv audit and check if 0 issues
Ci testing should run with fastembed again

@rileyok-ons
rileyok-ons requested a review from a team as a code owner November 6, 2026 16:37
@rileyok-ons rileyok-ons linked an issue Nov 6, 2026 that may be closed by this pull request
@github-actions github-actions Bot added the bug Something isn't working label Nov 6, 2026

@frayle-ons frayle-ons left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Running uv audit locally on this branch says:

"Found no known vulnerabilities and no adverse project statuses in 247 packages"

and i can see the fastembed vectoriser tests running on the CI unit test Job in the console output.

@rileyok-ons
rileyok-ons merged commit a69b7c7 into main Nov 6, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Re-add fastembed

2 participants