XLS-0094d-dynamic-MPT

[yinyiqian1]

discussion link: #289

[shawnxie999]

👍 looks good, left some nits

[mvadari]

@yinyiqian1 is this ready to merge? I can merge it if you address my comment.

[yinyiqian1]

@yinyiqian1 is this ready to merge? I can merge it if you address my comment.

@mvadari I just updated the spec. I added the link in discussion-from line, please let me know if you prefer changing it back to a raw address.
Other than that, the rest is ready to merge.

[mvadari]

@yinyiqian1 is this ready to merge? I can merge it if you address my comment.

@mvadari I just updated the spec. I added the link in discussion-from line, please let me know if you prefer changing it back to a raw address.
Other than that, the rest is ready to merge.

Raw address is preferable for now - that section is displayed with code formatting, so the link won't work.

[yinyiqian1]

@yinyiqian1 is this ready to merge? I can merge it if you address my comment.

@mvadari I just updated the spec. I added the link in discussion-from line, please let me know if you prefer changing it back to a raw address.
Other than that, the rest is ready to merge.

Raw address is preferable for now - that section is displayed with code formatting, so the link won't work.

Sure. I just changed it back.

[mvadari]

@yinyiqian1 please fix the linting check and it'll be good to merge (apologies for the addition in the checks, the repo is in some amount of flux at the moment)