Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
matYang created a review comment on a pull request on smartcontractkit/chainlink-ccip
`billing` is too broad, specifically, Commit is responsible for reporting gas prices, and token prices that don't come from keystone
matYang created a review comment on a pull request on smartcontractkit/chainlink-ccip
this would happen technically not at Commit but at msg execution right
matYang created a review comment on a pull request on smartcontractkit/chainlink-ccip
these contracts should be well-defined enough in other places, should be unnecessary to redefine them in billing doc
matYang created a review comment on a pull request on smartcontractkit/chainlink-ccip
`We have multiple contract interactions that happen`
matYang created a review comment on a pull request on smartcontractkit/chainlink-ccip
+1, source + dest is typically easier to understand
github-actions[bot] created a comment on a pull request on smartcontractkit/chainlink-ccip
| Metric | `observe-supported-chains` | `main` | |--|--|--| | **Coverage** | 72.4% | 72.4% |
asoliman92 pushed 1 commit to observe-supported-chains smartcontractkit/chainlink-ccip
- linting 468e4cb
asoliman92 opened a pull request on smartcontractkit/chainlink-ccip
FeeChain processor only observes supported chains
asoliman92 created a review comment on a pull request on smartcontractkit/chainlink-ccip
I think it's okay if it's a permanent link. contracts shouldn't be changing anyways.
matYang created a review comment on a pull request on smartcontractkit/chainlink-ccip
can we switch based on the first 4 bytes of selector and use the corresponding extra arg decoder version, as opposed to hardcoded v2; the v2 extra args selector is defined in `clients.sol`: ``` ...
matYang created a review comment on a pull request on smartcontractkit/chainlink-ccip
should reference the contracts in chainlink repo, afaik v2 code has been ported and actively developed there
github-actions[bot] created a comment on a pull request on smartcontractkit/chainlink-ccip
| Metric | `CCIP-3862` | `main` | |--|--|--| | **Coverage** | 72.4% | 72.5% |
rstout pushed 1 commit to CCIP-3862 smartcontractkit/chainlink-ccip
- Add gasLimit and tokenTransferGas to CalculateMessageMaxGas Parse gasLimit and tokenTransferGas from ccipocr3.Messag... 19ffc13
github-actions[bot] created a comment on a pull request on smartcontractkit/chainlink-ccip
| Metric | `CCIP-3862` | `main` | |--|--|--| | **Coverage** | 72.4% | 72.4% |
rstout pushed 1 commit to CCIP-3862 smartcontractkit/chainlink-ccip
- Add gasLimit and tokenTransferGas to CalculateMessageMaxGas Parse gasLimit and tokenTransferGas from ccipocr3.Messag... fbbd2eb
rstout pushed 1 commit to CCIP-3862 smartcontractkit/chainlink-ccip
- Add gasLimit and tokenTransferGas to CalculateMessageMaxGas Parse gasLimit and tokenTransferGas from ccipocr3.Messag... 77e2328
rstout pushed 15 commits to CCIP-3862 smartcontractkit/chainlink-ccip
- bump (#241) c4c6665
- RMN related fixes and more logs (#235) - fix a race condition - add more logs - fix stream name 753d327
- Migrate ccipocr3 package. (#240) Deprecating types in chainlink-common. 5b5f448
- Hardcoded RMN report version (#242) * hardcoded rmn report version 87a483a
- Fix: Token Price uses correct reader (#239) Fix: Token Price uses correct reader 3ec4824
- Use rmn report version digest as bytes (#243) 6757be1
- Detailed logging when binding addresses (#245) * Detailed logging when binding addresses --------- Co-authored... 8cc75f3
- discovery: Use getAllSourceChainConfigs (#227) 4ab500f
- Fix: OnRamp discovery (#249) Use another chainSelector other than self to get the destChainConfig Fix onRampDynamic... e5f3ecb
- Introducing GetAllConfigDigests (#253) b169beb
- Fix AddChainInbound (#254) Only observe supported chains Co-Authored: @austinborn Related chainlink PRs: 2736954
- Introduce unknown address type. (#248) e8564a2
- Switch integration branch back to develop. (#259) bcf311f
- Update link to new ccipocr3 location. (#252) 0a9dc6b
- Add gasLimit and tokenTransferGas to CalculateMessageMaxGas Parse gasLimit and tokenTransferGas from ccipocr3.Messag... 4d865c7
rstout opened a pull request on smartcontractkit/chainlink-ccip
Add gasLimit and tokenTransferGas to CalculateMessageMaxGas
Parse gasLimit and tokenTransferGas from ccipocr3.Message in CalculateMessageMaxGas and add these values to the gas estimate total.
