fix: deploy script validations (#667)

* fix: remove extra validation

* fix: use yarn pre-deploy in migrate command

Author: abarmat

package.json

@@ -82,7 +82,7 @@
     "build": "SKIP_LOAD=true scripts/build",
     "clean": "rm -rf build/ cache/ dist/",
     "compile": "hardhat compile",
-    "deploy": "yarn build && scripts/predeploy && hardhat migrate",
+    "deploy": "yarn build && yarn predeploy && hardhat migrate",
     "deploy-localhost": "yarn deploy --force --network localhost --graph-config config/graph.localhost.yml",
     "deploy-rinkeby": "yarn deploy --force --network rinkeby --graph-config config/graph.rinkeby.yml",
     "deploy-goerli": "yarn deploy --force --network goerli --graph-config config/graph.goerli.yml",

tasks/deployment/accounts.ts

@@ -6,10 +6,6 @@ import { updateItemValue, writeConfig } from '../../cli/config'
 task('migrate:accounts', '[localhost] Creates protocol accounts and saves them in graph config')
   .addParam('graphConfig', cliOpts.graphConfig.description, cliOpts.graphConfig.default)
   .setAction(async (taskArgs, hre) => {
-    if (hre.network.name !== 'localhost') {
-      throw new Error('This task can only be run on localhost network')
-    }
-
     const { graphConfig, getDeployer } = hre.graph({
       graphConfig: taskArgs.graphConfig,
     })

tasks/deployment/ownership.ts

@@ -9,10 +9,6 @@ task(
   .addParam('addressBook', cliOpts.addressBook.description, cliOpts.addressBook.default)
   .addParam('graphConfig', cliOpts.graphConfig.description, cliOpts.graphConfig.default)
   .setAction(async (taskArgs, hre) => {
-    if (hre.network.name !== 'localhost') {
-      throw new Error('This task can only be run on localhost network')
-    }
-
     const { contracts, getNamedAccounts } = hre.graph({
       addressBook: taskArgs.addressBook,
       graphConfig: taskArgs.graphConfig,