-
Notifications
You must be signed in to change notification settings - Fork 54
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
Report L1 base fee set by L2s #111
Merged
matYang
merged 34 commits into
ccip-develop
from
feature/offchain-report-l1-base-fee-from-l2
Oct 2, 2023
Merged
Changes from 15 commits
Commits
Show all changes
34 commits
Select commit
Hold shift + click to select a range
c35425e
CCIP plugin code sketch
matYang 0978d8f
Merge branch 'ccip-develop' into feature/offchain-report-l1-base-fee-…
matYang 27d328d
revert models changes
matYang 6fd2486
update gas price encoding and deviation
matYang 4288094
udpate exec cost estimation
matYang 65ec92f
Update exec cost estimation
matYang d68a786
update offchain config
matYang fb7b0af
use gas price struct instead of encoding where applicable
matYang b2c5866
do not use v2 suffix for latest version of offchain config
matYang 3985918
add nil check for gas update
matYang 159cc13
Merge branch 'ccip-develop' into feature/offchain-report-l1-base-fee-…
matYang bc94b46
Adding price estimator interfaces
matYang 78ab531
update commit plugin
matYang a6cd7c4
update name to prices
matYang 71992a3
update exec plugin
matYang 436eef3
comments and nits
matYang 5e4ec20
change the loaders to also return version
matYang 9b52b04
address comments
matYang 3367405
making tests build
matYang 5fb58a2
validate observed token prices match token with decimals
matYang 4bfb882
fixed commit reporting tests
matYang d3e7142
add test for validateObservations
matYang 37badf1
pass tests
matYang 76aa3ef
Merge branch 'ccip-develop' into feature/offchain-report-l1-base-fee-…
matYang 3d9d6ba
add test for exec getPrice
matYang 3237406
couple more tests in exec price estimator
matYang 337a715
complete exec price estimator test
matYang 9717f92
add da price estimator test
matYang d795be9
Merge branch 'ccip-develop' into feature/offchain-report-l1-base-fee-…
matYang c4f9718
resolve exec logpoller fiter test
matYang f619ede
address comments
matYang 42fa3bd
remove price estimator deviation and computeCost opts
matYang 9d7673f
Merge branch 'ccip-develop' into feature/offchain-report-l1-base-fee-…
matYang db7be89
fix 1.21 import
matYang File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe we should change the loaders to also return the version, as we already parse it in
LoadCommitStore