Silence more warnings (not all; some of them are going to be resolved in

other PRs)

src/governance/mock.rs

@@ -20,16 +20,16 @@ impl_outer_origin! {
 pub struct MockMembership {}
 impl<T: system::Trait> Members<T> for MockMembership {
     type Id = u32;
-    fn is_active_member(who: &T::AccountId) -> bool {
+    fn is_active_member(_who: &T::AccountId) -> bool {
         // all accounts are considered members.
         // There is currently no test coverage for non-members.
         // Should add some coverage, and update this method to reflect which accounts are or are not members
         true
     }
-    fn lookup_member_id(account_id: &T::AccountId) -> Result<Self::Id, &'static str> {
+    fn lookup_member_id(_account_id: &T::AccountId) -> Result<Self::Id, &'static str> {
         Err("not implemented!")
     }
-    fn lookup_account_by_member_id(id: Self::Id) -> Result<T::AccountId, &'static str> {
+    fn lookup_account_by_member_id(_id: Self::Id) -> Result<T::AccountId, &'static str> {
         Err("not implemented!")
     }
 }

src/governance/proposals.rs

@@ -623,16 +623,16 @@ mod tests {
     pub struct MockMembership {}
     impl<T: system::Trait> Members<T> for MockMembership {
         type Id = u32;
-        fn is_active_member(who: &T::AccountId) -> bool {
+        fn is_active_member(_who: &T::AccountId) -> bool {
             // all accounts are considered members.
             // There is currently no test coverage for non-members.
             // Should add some coverage, and update this method to reflect which accounts are or are not members
             true
         }
-        fn lookup_member_id(account_id: &T::AccountId) -> Result<Self::Id, &'static str> {
+        fn lookup_member_id(_account_id: &T::AccountId) -> Result<Self::Id, &'static str> {
             Err("not implemented!")
         }
-        fn lookup_account_by_member_id(id: Self::Id) -> Result<T::AccountId, &'static str> {
+        fn lookup_account_by_member_id(_id: Self::Id) -> Result<T::AccountId, &'static str> {
             Err("not implemented!")
         }
     }

src/migration.rs

@@ -7,8 +7,8 @@ use runtime_io::print;
 use crate::{VERSION};
 use crate::membership::members;
 use crate::roles::actors;
-use crate::governance::{GovernanceCurrency, BalanceOf };
-use runtime_primitives::traits::{Zero, Bounded, SimpleArithmetic, As};
+use crate::governance::{BalanceOf};
+use runtime_primitives::traits::{As};
 
 // When preparing a new major runtime release version bump this value to match it and update
 // the initialization code in runtime_initialization(). Because of the way substrate runs runtime code