Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

RunAll task for running on multiple networks #716

Draft
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

lilPlumberBoy
Copy link
Contributor

No description provided.

@codecov
Copy link

codecov bot commented Sep 14, 2022

Codecov Report

Merging #716 (529afb8) into master (9f02571) will increase coverage by 3.89%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #716      +/-   ##
==========================================
+ Coverage   87.41%   91.30%   +3.89%     
==========================================
  Files          43       43              
  Lines        2066     2071       +5     
  Branches      350      438      +88     
==========================================
+ Hits         1806     1891      +85     
+ Misses        260      180      -80     
Impacted Files Coverage Δ
contracts/xchainGauges/oracles/ChildOracle.sol 83.78% <0.00%> (+45.94%) ⬆️
...ontracts/xchainGauges/bridgers/ArbitrumBridger.sol 95.00% <0.00%> (+54.37%) ⬆️
contracts/xchainGauges/oracles/RootOracle.sol 100.00% <0.00%> (+54.83%) ⬆️
...ontracts/xchainGauges/bridgers/OptimismBridger.sol 100.00% <0.00%> (+100.00%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Copy link
Contributor

@penandlim penandlim left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Need to clean up imports via organize imports

tasks/runAll.ts Outdated
Comment on lines 21 to 22
console.log(`script failed on ${key} network with: `)
console.log(e)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this be console.error?

Comment on lines 23 to 40
// const poolRegistry = (await ethers.getContract(
// "PoolRegistry",
// )) as PoolRegistry

// const sdl = (await ethers.getContract("SDL")) as SDL
// const veSDL = (await ethers.getContract("VotingEscrow")) as VotingEscrow
// const minter = (await ethers.getContract("Minter")) as Minter
// const feeDistributor = (await ethers.getContract(
// "FeeDistributor",
// )) as FeeDistributor

// // get all pools from pool registry
// let poolRegLegnth: number = (await poolRegistry.getPoolsLength()).toNumber()
// const lastpool = await poolRegistry.getPoolDataAtIndex(poolRegLegnth - 1)
// console.log(lastpool)
// const DAY = 86400
// const WEEK = 86400 * 7
// const YEAR = WEEK * 52
Copy link
Contributor

@penandlim penandlim Sep 14, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please remove the comments if they are unnecessary to run

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants