Skip to content

Conversation

@arayikhalatyan
Copy link
Contributor

@arayikhalatyan arayikhalatyan commented May 22, 2025

All the integration tests should pass now. Merged with Native's PR so some of the tests could compile, I think for simplicity this can be merged after the native's PR.

@codspeed-hq
Copy link

codspeed-hq bot commented May 22, 2025

CodSpeed Walltime Performance Report

Merging #1679 will not alter performance

Comparing fix/integration-test (e6471a5) with feat/new-execution (cc1002c)

⚠️ Unknown Walltime execution environment detected

Using the Walltime instrument on standard Hosted Runners will lead to inconsistent data.

For the most accurate results, we recommend using CodSpeed Macro Runners: bare-metal machines fine-tuned for performance measurement consistency.

Summary

✅ 10 untouched benchmarks

@codspeed-hq
Copy link

codspeed-hq bot commented May 22, 2025

CodSpeed Instrumentation Performance Report

Merging #1679 will not alter performance

Comparing fix/integration-test (e6471a5) with feat/new-execution (cc1002c)

Summary

✅ 10 untouched benchmarks

@jonathanpwang
Copy link
Contributor

the tests involving native were infinite looping due to the bug @Golovanov399 is fixing

@jonathanpwang jonathanpwang merged commit b1387a3 into feat/new-execution May 22, 2025
19 of 25 checks passed
@jonathanpwang jonathanpwang deleted the fix/integration-test branch May 22, 2025 05:05
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.

4 participants