Skip to content

Testing FFI from C side#7126

Merged
myrrc merged 3 commits intodevelopfrom
myrrc/ffi-testing
Mar 23, 2026
Merged

Testing FFI from C side#7126
myrrc merged 3 commits intodevelopfrom
myrrc/ffi-testing

Conversation

@myrrc
Copy link
Contributor

@myrrc myrrc commented Mar 23, 2026

Add Catch2 target and first tests for C API

@myrrc myrrc requested a review from 0ax1 March 23, 2026 12:22
@myrrc myrrc force-pushed the myrrc/ffi-testing branch from 4f53f5a to d1bac0a Compare March 23, 2026 12:22
@myrrc myrrc added the changelog/chore A trivial change label Mar 23, 2026
@codspeed-hq
Copy link

codspeed-hq bot commented Mar 23, 2026

Merging this PR will not alter performance

✅ 1016 untouched benchmarks
⏩ 1522 skipped benchmarks1


Comparing myrrc/ffi-testing (f807e70) with develop (8fd83ab)

Open in CodSpeed

Footnotes

  1. 1522 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

@myrrc myrrc force-pushed the myrrc/ffi-testing branch from d1bac0a to 597df2c Compare March 23, 2026 12:30
Signed-off-by: Mikhail Kot <mikhail@spiraldb.com>
@myrrc myrrc force-pushed the myrrc/ffi-testing branch from 597df2c to 27d50cf Compare March 23, 2026 12:35
@myrrc myrrc enabled auto-merge (squash) March 23, 2026 13:17
Signed-off-by: Mikhail Kot <mikhail@spiraldb.com>
@myrrc myrrc requested a review from 0ax1 March 23, 2026 14:28
@myrrc myrrc requested a review from 0ax1 March 23, 2026 15:56
@myrrc myrrc disabled auto-merge March 23, 2026 16:02
Copy link
Contributor

@0ax1 0ax1 left a comment

Choose a reason for hiding this comment

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

lgtm after the last comments are addressed

Signed-off-by: Mikhail Kot <mikhail@spiraldb.com>
@myrrc myrrc enabled auto-merge (squash) March 23, 2026 16:08
@myrrc myrrc merged commit 6efc235 into develop Mar 23, 2026
56 checks passed
@myrrc myrrc deleted the myrrc/ffi-testing branch March 23, 2026 16:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog/chore A trivial change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants