Skip to content

[C API] CI for C API - #362

Merged
ethanglaser merged 9 commits into
dev/c-apifrom
dev/eglaser/c-api-ci
Aug 11, 2026
Merged

[C API] CI for C API#362
ethanglaser merged 9 commits into
dev/c-apifrom
dev/eglaser/c-api-ci

Conversation

@ethanglaser

@ethanglaser ethanglaser commented Aug 10, 2026

Copy link
Copy Markdown
Member

#360 reopened directly to C API branch

rfsaliev and others added 8 commits July 13, 2026 01:47
…nverter specializations

In namespace svs, partial specializations of lib::DispatchConverter with a
template parameter named 'Allocator' caused clang-15 (and clang-16) to
incorrectly resolve the name to the class template svs::lib::Allocator,
resulting in 'use of class template requires template arguments' errors.

Rename the template parameter from 'Allocator' to 'Alloc' in:
- bindings/c/src/data_builder/simple.hpp
- bindings/c/src/data_builder/sq.hpp
- bindings/c/src/data_builder/lvq.hpp
- bindings/c/src/data_builder/leanvec.hpp
…i-ci

# Conflicts:
#	bindings/cpp/CMakeLists.txt
@mergify

mergify Bot commented Aug 10, 2026

Copy link
Copy Markdown

⚠️ The sha of the head commit of this PR conflicts with #360. Mergify cannot evaluate rules on this PR. Once #360 is merged or closed, Mergify will resume processing this PR. ⚠️

@ethanglaser
ethanglaser marked this pull request as ready for review August 10, 2026 19:50
@mergify

mergify Bot commented Aug 10, 2026

Copy link
Copy Markdown

Tick the box to add this pull request to the merge queue (same as @mergifyio queue).

  • Queue this pull request

@ethanglaser ethanglaser changed the title Dev/eglaser/c api ci [C API] CI for C API Aug 10, 2026

@rfsaliev rfsaliev left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGFM

@ethanglaser
ethanglaser merged commit 08d9289 into dev/c-api Aug 11, 2026
22 checks passed
@ethanglaser
ethanglaser deleted the dev/eglaser/c-api-ci branch August 11, 2026 14:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants