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.

Sign up for GitHub

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

Jump to bottom

feat(GasEstimate): new base contract for gas estimation #133

Merged
merged 28 commits into from
Mar 16, 2024
Merged

feat(GasEstimate): new base contract for gas estimation #133

merged 28 commits into from
Mar 16, 2024

Conversation

re1ro
Copy link
Contributor

@re1ro re1ro commented Mar 13, 2024 •
edited by jira bot
Loading

  • GasEstimate: new contract for gas estimation logic
    AXE-3445

@re1ro re1ro self-assigned this Mar 13, 2024
@re1ro re1ro requested a review from a team as a code owner March 13, 2024 04:45
@re1ro re1ro mentioned this pull request Mar 13, 2024
Merged
5 tasks
milapsheth
milapsheth reviewed Mar 13, 2024
View reviewed changes
contracts/gas/GasEstimate.sol Outdated Show resolved Hide resolved
contracts/gas/GasEstimate.sol Outdated Show resolved Hide resolved
contracts/interfaces/IGasEstimate.sol Outdated Show resolved Hide resolved
contracts/gas/GasEstimate.sol Outdated Show resolved Hide resolved
re1ro and others added 13 commits March 13, 2024 13:06
@re1ro @milapsheth
Update contracts/gas/GasEstimate.sol
557de0f 
Co-authored-by: Milap Sheth <milap@axelar.network>
@re1ro @milapsheth
Update contracts/interfaces/IGasEstimate.sol
c200bc4 
Co-authored-by: Milap Sheth <milap@axelar.network>
@re1ro
feat(GasEstimate): using Optimism Ecotone gas model
42de430
@re1ro
feat(GasEstimate): using computed constants.
a82bb15
@re1ro
feat(GasEstimate): using computed constants.
324c8f3
@re1ro
feat(GasEstimate): adding GMP params to the estimation
f1b31f2
@re1ro
doc(GasEstimate): comment
bac19e6
@re1ro
doc(GasEstimate): comment
e696cca
@re1ro
test(GasEstimate): adding test contract
824b7b2
@re1ro
feat(IAxelarGasService): updating the interface with the latest
9ddb4b0
@re1ro
feat(IAxelarGasService): updating the interface with the latest
4b8eade
@re1ro
feat(IAxelarGasService): updating the interface with the latest
0249eb5
@re1ro
test(GasEstimate): adding test contract
62a07f4
milapsheth
milapsheth reviewed Mar 15, 2024
View reviewed changes
contracts/interfaces/IGasEstimate.sol Outdated Show resolved Hide resolved
contracts/interfaces/IGasEstimate.sol Outdated Show resolved Hide resolved
contracts/interfaces/IGasEstimate.sol Outdated Show resolved Hide resolved
contracts/test/gas/TestGasEstimate.sol Outdated Show resolved Hide resolved
contracts/interfaces/IGasEstimate.sol Outdated Show resolved Hide resolved
contracts/interfaces/IGasEstimate.sol Outdated Show resolved Hide resolved
contracts/gas/GasEstimate.sol Outdated Show resolved Hide resolved
re1ro and others added 5 commits March 15, 2024 19:52
@re1ro @milapsheth
Update contracts/interfaces/IGasEstimate.sol
520a82f 
Co-authored-by: Milap Sheth <milap@axelar.network>
@re1ro @milapsheth
Update contracts/interfaces/IGasEstimate.sol
27150eb 
Co-authored-by: Milap Sheth <milap@axelar.network>
@re1ro @milapsheth
Update contracts/interfaces/IGasEstimate.sol
e5b1ba3 
Co-authored-by: Milap Sheth <milap@axelar.network>
@re1ro
fix(GasEstimate): PR feedback
aae5ca9
@re1ro
refactor(GasEstimate): axelarBaseFee
0822133
milapsheth
milapsheth approved these changes Mar 16, 2024
View reviewed changes
contracts/interfaces/IInterchainGasEstimation.sol Outdated Show resolved Hide resolved
contracts/gas/InterchainGasEstimation.sol Outdated Show resolved Hide resolved
contracts/gas/InterchainGasEstimation.sol Outdated Show resolved Hide resolved
test/gas/InterchainGasEstimation.js Outdated Show resolved Hide resolved
test/gas/InterchainGasEstimation.js Outdated Show resolved Hide resolved
test/gas/InterchainGasEstimation.js Outdated Show resolved Hide resolved
test/gas/InterchainGasEstimation.js Outdated Show resolved Hide resolved
test/gas/InterchainGasEstimation.js Outdated Show resolved Hide resolved
@milapsheth
Copy link
Member

slither is failing

@milapsheth milapsheth merged commit 3e38026 into main Mar 16, 2024
4 checks passed
@milapsheth milapsheth deleted the feat/gas-estimate branch March 16, 2024 04:47
@re1ro re1ro mentioned this pull request Mar 20, 2024
Merged
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@milapsheth milapsheth milapsheth approved these changes

Assignees

@re1ro re1ro

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@re1ro @milapsheth