Skip to content

Commit

Permalink
Fix expose_rpc macro to handle all cases of generics from Runtime (#…
Browse files Browse the repository at this point in the history
…764)

* Start working on it

* Use different approach with phantom data, but...

Lifetime strikes back so far

* Start cleaning up and adding tests

* Adding more tests and cleaning up

* Adding some tests instead of removed ones

* Fix lint error

* Actually fix lint
  • Loading branch information
citizen-stig authored Sep 1, 2023
1 parent f21f011 commit ff8c800
Show file tree
Hide file tree
Showing 13 changed files with 489 additions and 477 deletions.
2 changes: 1 addition & 1 deletion Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ members = [

"utils/zk-cycle-macros",
"utils/zk-cycle-utils",

"module-system/sov-cli",
"module-system/sov-modules-stf-template",
"module-system/sov-modules-macros",
Expand Down
4 changes: 2 additions & 2 deletions examples/demo-stf/src/runtime.rs
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@ pub mod query {
/// instead of going through the DA layer.
#[cfg(not(feature = "experimental"))]
#[cfg_attr(feature = "native", derive(CliWallet), expose_rpc(DefaultContext))]
#[cfg_attr(feature = "native", derive(CliWallet), expose_rpc)]
#[derive(Genesis, DispatchCall, MessageCodec, DefaultRuntime)]
#[serialization(borsh::BorshDeserialize, borsh::BorshSerialize)]
#[cfg_attr(
Expand All @@ -84,7 +84,7 @@ pub struct Runtime<C: Context> {
}

#[cfg(feature = "experimental")]
#[cfg_attr(feature = "native", derive(CliWallet), expose_rpc(DefaultContext))]
#[cfg_attr(feature = "native", derive(CliWallet), expose_rpc)]
#[derive(Genesis, DispatchCall, MessageCodec, DefaultRuntime)]
#[serialization(borsh::BorshDeserialize, borsh::BorshSerialize)]
#[cfg_attr(
Expand Down
Loading

0 comments on commit ff8c800

Please sign in to comment.