ocurrent / current-bench

Experimental benchmarking infrastructure using OCurrent pipelines
Apache License 2.0
33 stars 17 forks source link

Cb check standalone #422

Closed ElectreAAS closed 1 year ago

ElectreAAS commented 1 year ago

This PR takes on the important parts of #416, but without the hassle of exposing cb-schema as a public toplevel library, which it turns out completely messes up our docker setup. Instead, it is now a library inside the pipeline, which is properly linked to and accessed when needed.

I also incorporated this comment: https://github.com/ocurrent/current-bench/pull/416#discussion_r1121924894

ElectreAAS commented 1 year ago

As the CI passes and the error reporting has been improved, I now declare cb-check to be in beta! I'll incorporate user-suggested changes and merge when satisfied

ElectreAAS commented 1 year ago

The force-pushes were rebases and a clean-up of the commit history, without changes. As everyone seems happy, I'll merge as-is.