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

Pass the home chain selector #14500

Merged
merged 12 commits into from
Sep 30, 2024
Merged

Pass the home chain selector #14500

merged 12 commits into from
Sep 30, 2024

Conversation

0xnogo
Copy link
Collaborator

@0xnogo 0xnogo commented Sep 19, 2024

Passing the home chain selector so that this commit plugin can construct the rmn home contract binding. We also added RMNHome as part of the home chain reader config.

This PR will not be pushed as-is as it is relying on a non main branch commit.

The commit of smartcontractkit/chainlink-ccip#140 will be used for chainlink-ccip dependency.

@0xnogo 0xnogo requested review from a team as code owners September 19, 2024 19:09
@0xnogo 0xnogo requested review from ilija42 and removed request for a team September 19, 2024 19:09
ilija42
ilija42 previously approved these changes Sep 20, 2024
@0xnogo 0xnogo changed the title pass the home chain selector Pass the home chain selector Sep 25, 2024
@0xnogo 0xnogo marked this pull request as ready for review September 26, 2024 10:44
@0xnogo 0xnogo added this pull request to the merge queue Sep 30, 2024
Merged via the queue into develop with commit d435981 Sep 30, 2024
147 checks passed
@0xnogo 0xnogo deleted the ng/pass-home-chain-selector branch September 30, 2024 08:29
AnieeG pushed a commit that referenced this pull request Sep 30, 2024
* pass the home chain selector

* Refactor contract reader configuration and add RMNHome contract support

* Update chainlink-ccip dependency version

* changeset

* new chainlink-ccip version

* bump chainlink-ccip

* bump to main branch commit

* using latest chainlink-ccip commit
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.

4 participants