search

dylibso / observe-sdk

Continuous runtime observablity SDKs to monitor WebAssembly code.
https://dev.dylibso.com/docs/observe/overview
Apache License 2.0
156 stars 7 forks source link

feat(observe api): seperate c++ bindings from c bindings #168

Closed G4Vi closed 1 month ago

G4Vi commented 1 month ago

This allows using the c++ bindings without the c bindings. Making the C++ bindings easier to use also a single header library. In a C++ project both bindings still may be used all the time.

nilslice commented 1 month ago

Should we move the component stuff out of here now that we've provided it to the community in wasi-observe? Seems like its fallen behind tooling and causing CI to fail.

G4Vi commented 1 month ago

Should we move the component stuff out of here now that we've provided it to the community in wasi-observe? Seems like its fallen behind tooling and causing CI to fail.

I'd be happy to nuke the observe component support as I don't think anyone uses it and fixing it up, is probably just as much work as readding it, as I'm sure there's been tooling changes. If we have a need for it or someone contributes it, we could always readd it. @chrisdickinson What do you think?

chrisdickinson commented 1 month ago

Yeah, probably worth ripping that out for now. Git will remember it for us if we ever need it in the future!