Skip to content

Refine language and clarity in README.md#46

Merged
beihaili merged 1 commit into
beihaili:mainfrom
isogay:patch-3
Mar 26, 2026
Merged

Refine language and clarity in README.md#46
beihaili merged 1 commit into
beihaili:mainfrom
isogay:patch-3

Conversation

@isogay

@isogay isogay commented Mar 24, 2026

Copy link
Copy Markdown
Contributor

Updated the README to enhance clarity and improve phrasing throughout the document, including transaction descriptions and historical references.

变更描述

变更类型

  • Bug 修复
  • 新功能
  • 文档改进
  • 代码重构
  • 测试
  • 其他

相关 Issue

Closes #

测试

  • 本地测试通过 (npm test)
  • Lint 检查通过 (npm run lint)
  • 构建成功 (npm run build)

截图(如适用)

检查清单

  • 代码遵循项目的代码风格
  • 已添加必要的测试
  • 所有现有测试通过
  • 已更新相关文档

Updated the README to enhance clarity and improve phrasing throughout the document, including transaction descriptions and historical references.

@beihaili beihaili left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Minor issue: "Corresponds" should stay "Correspond" (subject "Easter Eggs" is plural). Otherwise looks good — please fix and I will merge.

@beihaili

Copy link
Copy Markdown
Owner

Almost there @isogay! Just the one grammar fix as mentioned:

In the overview table, the "Input Amount Easter Eggs" row:

  • Current: Corresponds to historical events and vulnerability IDs
  • Should be: Correspond to historical events and vulnerability IDs

("Easter Eggs" is plural → "Correspond" not "Corresponds")

The "Block Height Meme" row with Corresponds to the 850,000 BTC is correct as-is (singular reference).

Everything else looks great — the phrasing improvements are solid. Fix this one line and I'll merge. Thanks!

@isogay

isogay commented Mar 25, 2026

Copy link
Copy Markdown
Contributor Author

Almost there @isogay! Just the one grammar fix as mentioned:

In the overview table, the "Input Amount Easter Eggs" row:

  • Current: Corresponds to historical events and vulnerability IDs
  • Should be: Correspond to historical events and vulnerability IDs

("Easter Eggs" is plural → "Correspond" not "Corresponds")

The "Block Height Meme" row with Corresponds to the 850,000 BTC is correct as-is (singular reference).

Everything else looks great — the phrasing improvements are solid. Fix this one line and I'll merge. Thanks!

sure

@beihaili
beihaili merged commit c8c7113 into beihaili:main Mar 26, 2026
1 check failed
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