Closed ngundotra closed 1 year ago
good to merge?
This LGTM. It's mostly expressing the current anchor IDL - which is fine. Once it's merged I can add/adapt some types to find a common ground so shank/solita types are reflected as well.
One thing we can also discuss is remove some parts that are too anchor specific, (maybe events). Solita doesn't have those for instance. Basically the idea would be to end up with something like the following:
In the above events would just be an extension that anchor adds on top of the core spec.
However we can merge this for now IMO and discuss what should go/stay in core separately.
We should split them up so it's easier to comment on them & make changes.
Apologies @thlorenz I think this will require an additional step in the build command.