chore: cleanup some redundant comments

Author: tamtamchik

lib/protocol/networks.ts

@@ -6,7 +6,6 @@ import { log } from "lib";
 
 import { ProtocolNetworkItems } from "./types";
 
-// If we are running in Hardhat without fork, we need to deploy the contracts from scratch deploy first to run integration tests
 export function isNonForkingHardhatNetwork() {
   const networkName = hre.network.name;
   if (networkName === "hardhat") {

lib/state-file.ts

@@ -191,7 +191,6 @@ export async function resetStateFile(networkName: string = hardhatNetwork.name):
   }
 }
 
-// If network name is undefined current hardhat network will be used
 export function persistNetworkState(state: DeploymentState, networkName: string = hardhatNetwork.name): void {
   const fileName = _getFileName(networkName, NETWORK_STATE_FILE_BASENAME, NETWORK_STATE_FILE_DIR);
   const stateSorted = _sortKeysAlphabetically(state);

scripts/dao-deploy.sh

@@ -2,9 +2,6 @@
 set -e +u
 set -o pipefail
 
-# TODO: Do we still need to set these variable?
-# ARAGON_APPS_REPO_REF=import-shared-minime
-
 # Check for required environment variables
 if [[ -z "${DEPLOYER}" ]]; then
   echo "Error: Environment variable DEPLOYER must be set"

scripts/dao-upgrade.sh

@@ -2,9 +2,6 @@
 set -e +u
 set -o pipefail
 
-# TODO: Do we still need to set these variable?
-# ARAGON_APPS_REPO_REF=import-shared-minime
-
 # Check for required environment variables
 if [[ -z "${DEPLOYER}" ]]; then
   echo "Error: Environment variable DEPLOYER must be set"