Skip to content

Conversion tool improvement - #13

Closed
tea2x wants to merge 9 commits into
nervosnetwork:mainfrom
tea2x:main
Closed

Conversion tool improvement#13
tea2x wants to merge 9 commits into
nervosnetwork:mainfrom
tea2x:main

Conversation

@tea2x

@tea2x tea2x commented Jan 11, 2025

Copy link
Copy Markdown
Contributor

Great project! This PR contains some improvements:

  • Conversion/migrate tool (make tool cli accept standard 256bit digest hex string).
  • add rust workspace configuration so rust analyzer + other rust tools work.
  • improve make build process so later builds will update previous builds.
  • update readme & correct test-net deployment info.

@xxuejie

xxuejie commented Feb 21, 2025

Copy link
Copy Markdown
Contributor

Hey thanks for this! I do want to mention that I'm rearchitecting this lock these days, feel free to check out #14 for progresses.

Unfortunately, some of our work overlap(e.g., Rust related part), what do you say we revisit this PR when the overhaul is done?

@tea2x

tea2x commented Feb 21, 2025

Copy link
Copy Markdown
Contributor Author

Hi @xxuejie it's nothing.

what do you say we revisit this PR when the overhaul is done?

Yes please!

@tea2x

tea2x commented Feb 21, 2025

Copy link
Copy Markdown
Contributor Author

it's actually not much in this PR so you might want to just update directly in #14

@tea2x tea2x closed this Aug 29, 2025
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