Skip to content

Commit

Permalink
Remove redundant deployments (#1014)
Browse files Browse the repository at this point in the history
  • Loading branch information
jtriley2p authored Aug 5, 2022
1 parent 9e7fa81 commit ab0fdb4
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 22 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/call.test-hardhat-deployer.yml
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ jobs:
working-directory: ${{ env.contracts-working-directory }}

- name: "Install dependencies"
run: yarn
run: yarn install --frozen-lockfile
working-directory: ${{ env.hardhat-working-directory }}

- name: "Run test suite"
Expand Down
25 changes: 4 additions & 21 deletions packages/hardhat-deployer/src/SuperfluidFrameworkDeployer.ts
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
import ERC20PresetMinterPauser from "@superfluid-finance/ethereum-contracts/build/contracts/ERC20PresetMinterPauser.json";
import Resolver from "@superfluid-finance/ethereum-contracts/build/contracts/Resolver.json";
import SlotsBitmapLibrary from "@superfluid-finance/ethereum-contracts/build/contracts/SlotsBitmapLibrary.json";
import SuperToken from "@superfluid-finance/ethereum-contracts/build/contracts/SuperToken.json";
import SuperfluidFrameworkDeployerContract from "@superfluid-finance/ethereum-contracts/build/contracts/SuperfluidFrameworkDeployer.json";
Expand Down Expand Up @@ -51,13 +50,6 @@ export class SuperfluidFrameworkDeployer {
await deployer.provider!.sendTransaction(ERC1820_PAYLOAD);
}

// Deploy `Resolver`
const resolver = await new ethers.ContractFactory(
Resolver.abi,
Resolver.bytecode,
deployer
).deploy(overrides);

const slotsBitmapLibrary = await new ethers.ContractFactory(
SlotsBitmapLibrary.abi,
SlotsBitmapLibrary.bytecode,
Expand All @@ -75,24 +67,15 @@ export class SuperfluidFrameworkDeployer {
deployer
).deploy({ gasLimit: 100000000 });

const framework = await superfluidFrameworkDeployer.getFramework();

// Register `TestGovernance` with `Resolver`
await resolver.set(
"TestGovernance.test",
framework.governance,
overrides
);

// // Register `Superfluid` with `Resolver`
await resolver.set("Superfluid.test", framework.host, overrides);
const { resolver: resolverAddress } =
await superfluidFrameworkDeployer.getFramework();

// Update Internal State
this.address = superfluidFrameworkDeployer.address;
this.resolverAddress = resolver.address;
this.resolverAddress = resolverAddress;

// Return `Resolver` address
return resolver.address;
return resolverAddress;
}

async deployWrapperSuperToken(
Expand Down

0 comments on commit ab0fdb4

Please sign in to comment.