Skip to content

test(autobahn): drop stale -b block TODO#3454

Open
wen-coding wants to merge 1 commit into
mainfrom
wen/drop_bblock_todo
Open

test(autobahn): drop stale -b block TODO#3454
wen-coding wants to merge 1 commit into
mainfrom
wen/drop_bblock_todo

Conversation

@wen-coding
Copy link
Copy Markdown
Contributor

We don't intend to support `-b block` on autobahn. `-b sync` is the path forward; the migration in #3363 + #3406 makes that explicit on the hardhat side too.

We don't intend to support -b block on autobahn. -b sync is the path
forward.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@cursor
Copy link
Copy Markdown

cursor Bot commented May 17, 2026

PR Summary

Low Risk
Comment-only change in an integration test; no runtime logic or behavior is modified.

Overview
Removes the outdated comment/TODO in integration_test/autobahn/autobahn_test.go implying future support for -b block in Autobahn mode, clarifying that the bank transfer test intentionally uses -b sync.

Reviewed by Cursor Bugbot for commit 142affe. Bugbot is set up for automated code reviews on this repo. Configure here.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 17, 2026

The latest Buf updates on your PR. Results from workflow Buf / buf (pull_request).

BuildFormatLintBreakingUpdated (UTC)
✅ passed✅ passed✅ passed✅ passedMay 17, 2026, 6:27 PM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant