Ashok Menon
Ashok Menon
(Actually, it's now back to you to undo the `u128` support -- the tests look good, I was just looking at an old copy of the PR for some reason).
Ah, @3rror404, this change requires updating the "empty genesis snapshot" (basically it changes the initial state of the chain). You can do this by running `cargo simtest` which will initially...
It looks like some of the other snapshots need to be updated as well. The steps I shared in my previous comment should work for those too, but you may...
Let's see what the CI says -- I think I saw some other snapshots that needed updating (they are to do with the gas schedule and how much operations cost)...
So, yep, CI tells us that there are some more snapshots to refresh. I believe they are runnable on your side with `cargo nextest run --no-fail-fast`. Thanks so much for...
I believe you've already updated the `sui-genesis` one (that was the first one you did). Let's see what CI says!
I believe you've correctly updated all the necessary snapshots now, but unfortunately they now conflict, so you'll need to rebase, and then re-run the snapshot commands. For that, following @oxade's...
Excellent job @3rror404, thanks for sticking with it!
The example looks good! We should likely document it [here](https://examples.sui.io/basics/move-toml.html) as well. I'll leave it to @randall-Mysten to give the final stamp from a docs perspective.
Checking back in on this as this week's oncall, @velvia and @longbowlu, can we close this now?