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

CCIP-4329 modify setChainRateLimiterConfig file for setting multiple … #1569

Merged
merged 3 commits into from
Dec 10, 2024

Conversation

RensR
Copy link
Collaborator

@RensR RensR commented Dec 10, 2024

…rate limits (#15386)

  • modify setChainRateLimiterConfig file for setting multiple rate limits

  • Update gethwrappers

  • [Bot] Update changeset file with jira issues

  • snapshot fix

  • fix wrappers after merge

  • fix test that was broken in merge

  • fix liquidityManager snapshot which is affected by token pool changes

  • fix broken integration tests from removing critical setup function

  • add array length check for setChainRateLimitConfigs

  • formatting and snapshot fix

  • Update gethwrappers

  • simplify test setups

  • Update gethwrappers

  • update wrappers and snapshots which were broken in merge

  • Update gethwrappers

  • update wrappers and snapshot

  • changeset

  • snapshot update

  • Delete contracts/.changeset/orange-buckets-live.md

  • attempt fix changeset

  • attempt changeset fix with update pnpm package

  • re-add function that was removed early in PR to prevent any required tooling changes

  • update liquidityManager snapshot because of changes to token pools


Motivation

Solution

…rate limits (#15386)

* modify setChainRateLimiterConfig file for setting multiple rate limits

* Update gethwrappers

* [Bot] Update changeset file with jira issues

* snapshot fix

* fix wrappers after merge

* fix test that was broken in merge

* fix liquidityManager snapshot which is affected by token pool changes

* fix broken integration tests from removing critical setup function

* add array length check for setChainRateLimitConfigs

* formatting and snapshot fix

* Update gethwrappers

* simplify test setups

* Update gethwrappers

* update wrappers and snapshots which were broken in merge

* Update gethwrappers

* update wrappers and snapshot

* changeset

* snapshot update

* Delete contracts/.changeset/orange-buckets-live.md

* attempt fix changeset

* attempt changeset fix with update pnpm package

* re-add function that was removed early in PR to prevent any required tooling changes

* update liquidityManager snapshot because of changes to token pools

---------

Co-Authored-By: app-token-issuer-infra-releng[bot] <120227048+app-token-issuer-infra-releng[bot]@users.noreply.github.com>
@RensR RensR requested review from makramkd, elatoskinas, RayXpub and a team as code owners December 10, 2024 12:50
@RensR RensR requested review from winder, rstout, asoliman92, dimkouv, mateusz-sekara and 0xAustinWang and removed request for a team December 10, 2024 12:50
Copy link
Contributor

Static analysis results are available

Hey @RensR, you can view Slither reports in the job summary here or download them as artifact here.

Please check them before merging and make sure you have addressed all issues.

@jhweintraub jhweintraub merged commit 478f0e5 into release/contracts-ccip-1.5.1 Dec 10, 2024
162 of 167 checks passed
@jhweintraub jhweintraub deleted the multi-rl-config branch December 10, 2024 16:07
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.

3 participants