Skip to content

Remove ICS precompile#247

Merged
Thaleszh merged 1 commit into
mainfrom
fix/remove-ics
Jan 22, 2026
Merged

Remove ICS precompile#247
Thaleszh merged 1 commit into
mainfrom
fix/remove-ics

Conversation

@Thaleszh

Copy link
Copy Markdown
Contributor

Description

Removes and disables ICS precompile

Type of change

  • Bug fix (non-breaking change which fixes an issue)

@Thaleszh Thaleszh requested a review from jhelison as a code owner January 22, 2026 18:31
@coderabbitai

coderabbitai Bot commented Jan 22, 2026

Copy link
Copy Markdown
Contributor

Walkthrough

The pull request removes the IBC ICS20 precompile from the codebase. Changes include: deletion of the ICS20 precompile import and initialization from the keepers module, addition of a changelog entry documenting the removal, and implementation of upgrade logic in v7_0 that filters the ICS precompile address (0x0000000000000000000000000000000000000802) from the EVM module's active static precompiles and updates the stored EVM parameters with the filtered list.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Remove ICS precompile' accurately and concisely describes the main change across all modified files.
Description check ✅ Passed The description directly relates to the changeset, stating the intent to remove and disable the ICS precompile.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@Thaleszh Thaleszh merged commit 849a1ec into main Jan 22, 2026
9 checks passed
@Thaleszh Thaleszh deleted the fix/remove-ics branch January 22, 2026 19:11
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.

2 participants