Use non-binary pronouns in comments. (#13209)

* use non binary pronouns in comments

* cargo fmt

* fix the use of "it" with "they" when dealing about an opperations identity

substrate/frame/contracts/src/exec.rs

@@ -2056,9 +2056,9 @@ mod tests {
 	#[test]
 	fn code_hash_returns_proper_values() {
 		let code_bob = MockLoader::insert(Call, |ctx, _| {
-			// ALICE is not a contract and hence she does not have a code_hash
+			// ALICE is not a contract and hence they do not have a code_hash
 			assert!(ctx.ext.code_hash(&ALICE).is_none());
-			// BOB is a contract and hence he has a code_hash
+			// BOB is a contract and hence it has a code_hash
 			assert!(ctx.ext.code_hash(&BOB).is_some());
 			exec_success()
 		});

substrate/frame/elections-phragmen/src/benchmarking.rs

@@ -32,7 +32,7 @@ const BALANCE_FACTOR: u32 = 250;
 /// grab new account with infinite balance.
 fn endowed_account<T: Config>(name: &'static str, index: u32) -> T::AccountId {
 	let account: T::AccountId = account(name, index, 0);
-	// Fund each account with at-least his stake but still a sane amount as to not mess up
+	// Fund each account with at-least their stake but still a sane amount as to not mess up
 	// the vote calculation.
 	let amount = default_stake::<T>(T::MaxVoters::get()) * BalanceOf::<T>::from(BALANCE_FACTOR);
 	let _ = T::Currency::make_free_balance_be(&account, amount);

substrate/frame/nfts/src/features/transfer.rs

@@ -64,7 +64,7 @@ impl<T: Config<I>, I: 'static> Pallet<T, I> {
 		details.owner = dest;
 
 		// The approved accounts have to be reset to None, because otherwise pre-approve attack
-		// would be possible, where the owner can approve his second account before making the
+		// would be possible, where the owner can approve their second account before making the
 		// transaction and then claiming the item back.
 		details.approvals.clear();
 

substrate/frame/nomination-pools/src/migration.rs

@@ -63,7 +63,7 @@ pub mod v1 {
 
 	/// Trivial migration which makes the roles of each pool optional.
 	///
-	/// Note: The depositor is not optional since he can never change.
+	/// Note: The depositor is not optional since they can never change.
 	pub struct MigrateToV1<T>(sp_std::marker::PhantomData<T>);
 	impl<T: Config> OnRuntimeUpgrade for MigrateToV1<T> {
 		fn on_runtime_upgrade() -> Weight {

substrate/frame/nomination-pools/test-staking/src/lib.rs

@@ -531,7 +531,7 @@ fn pool_slash_proportional() {
 
 #[test]
 fn pool_slash_non_proportional_only_bonded_pool() {
-	// A typical example where a pool member unbonds in era 99, and he can get away with a slash
+	// A typical example where a pool member unbonds in era 99, and they can get away with a slash
 	// that happened in era 100, as long as the pool has enough active bond to cover the slash. If
 	// everything else in the slashing/staking system works, this should always be the case.
 	// Nonetheless, `ledger.slash` has been written such that it will slash greedily from any chunk

substrate/frame/root-offences/src/tests.rs

@@ -44,7 +44,7 @@ fn create_offence_works_given_root_origin() {
 		// the slash should be applied right away.
 		assert_eq!(Balances::free_balance(11), 500);
 
-		// the other validator should keep his balance, because we only created
+		// the other validator should keep their balance, because we only created
 		// an offences for the first validator.
 		assert_eq!(Balances::free_balance(21), 1000);
 	})
@@ -68,7 +68,7 @@ fn create_offence_wont_slash_non_active_validators() {
 		// so 31 didn't get slashed.
 		assert_eq!(Balances::free_balance(31), 500);
 
-		// but 11 is an active validator so he got slashed.
+		// but 11 is an active validator so they got slashed.
 		assert_eq!(Balances::free_balance(11), 800);
 	})
 }

substrate/frame/staking/src/tests.rs

@@ -1901,7 +1901,7 @@ fn wrong_vote_is_moot() {
 #[test]
 fn bond_with_no_staked_value() {
 	// Behavior when someone bonds with no staked value.
-	// Particularly when she votes and the candidate is elected.
+	// Particularly when they votes and the candidate is elected.
 	ExtBuilder::default()
 		.validator_count(3)
 		.existential_deposit(5)

substrate/frame/uniques/src/functions.rs

@@ -50,8 +50,8 @@ impl<T: Config<I>, I: 'static> Pallet<T, I> {
 		details.owner = dest;
 
 		// The approved account has to be reset to None, because otherwise pre-approve attack would
-		// be possible, where the owner can approve his second account before making the transaction
-		// and then claiming the item back.
+		// be possible, where the owner can approve their second account before making the
+		// transaction and then claiming the item back.
 		details.approved = None;
 
 		Item::<T, I>::insert(&collection, &item, &details);