Skip to content

Add test that verifies embeddings aren't supported#43

Merged
kzu merged 1 commit intomainfrom
dev/no-embeddings
Jan 5, 2026
Merged

Add test that verifies embeddings aren't supported#43
kzu merged 1 commit intomainfrom
dev/no-embeddings

Conversation

@kzu
Copy link
Member

@kzu kzu commented Jan 5, 2026

xAI currently returns no embedding models, which is the reason we don't provide an IEmbeddingGenerator implementation. If this changes in the future, this new test will fail and we can fix that by providing an implementation then.

xAI currently returns no embedding models, which is the reason we don't provide an IEmbeddingGenerator implementation. If this changes in the future, this new test will fail and we can fix that by providing an implementation then.
@kzu kzu added the tech debt label Jan 5, 2026
@kzu kzu enabled auto-merge (rebase) January 5, 2026 16:09
@kzu
Copy link
Member Author

kzu commented Jan 5, 2026

21 passed 21 passed 12 skipped

🧪 Details on Ubuntu 24.04.3 LTS

from retest v1.1.0 on .NET 10.0.1 with 💜 by @devlooped

@kzu kzu merged commit 612ed9c into main Jan 5, 2026
7 checks passed
@kzu kzu deleted the dev/no-embeddings branch January 5, 2026 16:12
@devlooped devlooped locked and limited conversation to collaborators Feb 5, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant