Conversation
2a0227b to
19dceb0
Compare
ebma
left a comment
There was a problem hiding this comment.
It looks good overall. Hard to say if everything is correct since it's just so much that changed. Thanks, @b-yap for commenting with code links on the more critical/confusing parts 👍 I marked most of them as resolved to indicate acknowledgment. Although maybe I should have waited for fellow reviewers to also read them before doing that.
ebma
left a comment
There was a problem hiding this comment.
Thanks for all your work @b-yap. I re-reviewed and it looks good to me. Although it's quite a lot of changes and hard to grasp all of it.
Just a question regarding the benchmarks. It's great that we have them, and also for the other pallets. Was this a requirement by the new Polkadot version that the WeightInfo field cannot be empty?
742f7a7 to
4dca813
Compare
4dca813 to
09176d4
Compare
ebma
left a comment
There was a problem hiding this comment.
I closed the unresolved conversations now and will approve and merge.
closes #219
not yet ready! Just visibility on the changes I've done so far. Added comments on where I got the changes too.
Update:
development-runtimeshould compile fine.foucoco-runtimeamplitude-runtimependulum-runtime