Address review grumbles from #99

Cargo.toml

@@ -17,8 +17,7 @@ keywords = ["parity", "substrate", "blockchain"]
 include = ["Cargo.toml", "src/**/*.rs", "README.md", "LICENSE"]
 
 [features]
-kusama = []
-default = ["kusama"]
+default = []
 client = ["substrate-subxt-client"]
 
 # enable this feature to run tests which require a local dev chain node

README.md

@@ -6,9 +6,6 @@ A library to **sub**mit e**xt**rinsics to a [substrate](https://github.com/parit
 
 See [examples](./examples).
 
-If you use `#[derive(Call)]` without `#[module]` in the same module, you will get errors
-complaining about an undefined method with a name starting with `with_`.
-
 **Alternatives**
 
 [substrate-api-client](https://github.com/scs/substrate-api-client) provides similar functionality.

proc-macro/src/lib.rs

@@ -133,6 +133,10 @@ decl_derive!(
     ///
     /// Under the hood the implementation calls [submit()](../substrate_subxt/struct.Client.html#method.submit) and
     /// [watch()](../substrate_subxt/struct.Client.html#method.watch) respectively.
+    ///
+    /// *N.B.* You must use the `#[derive(Call)]` macro with `#[module]` in the same module or you will get errors
+    /// about undefined method with a name starting with `with_`.
+
     #[proc_macro_error] call
 );
 fn call(s: Structure) -> TokenStream {

src/frame/system.rs

@@ -99,10 +99,7 @@ pub trait System {
     type AccountId: Parameter
         + Member
         + MaybeSerialize
-        + Debug
         + MaybeDisplay
-        + Encode
-        + Decode
         + Ord
         + Default;
 

src/runtimes.rs

@@ -52,7 +52,6 @@ impl sp_runtime::BoundToRuntimeAppPublic for Grandpa {
 
 use sp_authority_discovery::AuthorityId as AuthorityDiscoveryId;
 
-#[cfg(feature = "kusama")]
 mod validator_app {
     use application_crypto::{
         app_crypto,
@@ -62,11 +61,9 @@ mod validator_app {
 }
 
 /// Parachain marker struct
-#[cfg(feature = "kusama")]
 #[derive(Copy, Clone, Debug, Default, Eq, PartialEq, Ord, PartialOrd, Hash)]
 pub struct Parachains;
 
-#[cfg(feature = "kusama")]
 impl sp_runtime::BoundToRuntimeAppPublic for Parachains {
     type Public = validator_app::Public;
 }