Skip to content

feat: foreign chain inspector for abstract block chain#2015

Merged
gilcu3 merged 24 commits intomainfrom
2011-create-foreign-chain-inspector-for-abstract-block-chain
Feb 10, 2026
Merged

feat: foreign chain inspector for abstract block chain#2015
gilcu3 merged 24 commits intomainfrom
2011-create-foreign-chain-inspector-for-abstract-block-chain

Conversation

@DSharifi
Copy link
Contributor

@DSharifi DSharifi commented Feb 9, 2026

closes #2011

Adds a new inspector for the abstract block-chain.

@DSharifi DSharifi linked an issue Feb 9, 2026 that may be closed by this pull request
@DSharifi DSharifi changed the base branch from main to dsharifi/support-json-rpc-v2-instead February 9, 2026 09:38
Base automatically changed from dsharifi/support-json-rpc-v2-instead to main February 9, 2026 12:58
@DSharifi DSharifi force-pushed the 2011-create-foreign-chain-inspector-for-abstract-block-chain branch from c3e19e7 to 012c59f Compare February 9, 2026 17:21
@DSharifi DSharifi marked this pull request as ready for review February 9, 2026 21:40
@DSharifi DSharifi requested review from gilcu3 and netrome February 9, 2026 21:40
@DSharifi DSharifi self-assigned this Feb 9, 2026
gilcu3
gilcu3 previously approved these changes Feb 10, 2026
Copy link
Contributor

@gilcu3 gilcu3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you, LGTM!

Copy link
Collaborator

@netrome netrome left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Started reviewing. Will get back to it after lunch.

netrome
netrome previously approved these changes Feb 10, 2026
Copy link
Collaborator

@netrome netrome left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some more nits/naming thoughts but otherwise this looks good to me!

@DSharifi DSharifi requested review from gilcu3 and netrome February 10, 2026 13:40
netrome
netrome previously approved these changes Feb 10, 2026
@DSharifi DSharifi enabled auto-merge February 10, 2026 14:32
@DSharifi DSharifi added this pull request to the merge queue Feb 10, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 10, 2026
@gilcu3 gilcu3 added this pull request to the merge queue Feb 10, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 10, 2026
@gilcu3 gilcu3 added this pull request to the merge queue Feb 10, 2026
Merged via the queue into main with commit dab2d73 Feb 10, 2026
9 checks passed
@gilcu3 gilcu3 deleted the 2011-create-foreign-chain-inspector-for-abstract-block-chain branch February 10, 2026 15:42
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.

create foreign chain inspector for Abstract block chain

3 participants