[pallet-revive] impl_revive_api macro

[pgherveou]

Move pallet-revive runtime api implementation in a macro, so that we don't repeat the code for every runtime.

[pgherveou]

/cmd prdoc --audience runtime_dev --bump patch

[xermicus]

Dumb question: Instead of a macro, can't we just provide default implementations for the trait methods?

[pgherveou]

Dumb question: Instead of a macro, can't we just provide default implementations for the trait methods?

I don't think that sp_api::decl_runtime_apis! accept it
You also need to access some concrete type from the runtime to implement it

[xermicus]

Yeah I am not sure. You might be right. I just glanced at the first implementation which doesn't seem to have any generics in it's signature.

[paritytech-workflow-stopper]

All GitHub workflows were cancelled due to failure one of the required jobs.
Failed workflow url: https://github.com/paritytech/polkadot-sdk/actions/runs/15254950229
Failed job name: test-linux-stable

[pgherveou]

mmm compile but does not seem to work :/

Error: ErrorObject { code: ServerError(-32000), message: "RPC error: ErrorObject { code: ServerError(4003), message: \"Client error: Execution failed: Other: Exported method ReviveApi_balance is not found\", data: None }", data: None }

will check this out

[pgherveou]

/cmd fmt

[athei]

Yeah I am not sure. You might be right. I just glanced at the first implementation which doesn't seem to have any generics in it's signature.

What do you mean? Our RuntimeAPI need at least the Balance and AccountId which are generic.