Closed nrc closed 2 years ago
Awesome :) I'll try to get to reviewing this soon.
There's also either an issue or checkbox somewhere that needs to be changed after this is merged 😉
@bors delegate+
:v: @nrc can now approve this pull request
@bors r+
:pushpin: Commit e92db1dfb4661b6a78c70c3f86c5e089e7879f2c has been approved by nrc
:hourglass: Testing commit e92db1dfb4661b6a78c70c3f86c5e089e7879f2c with merge 4b95a894b8395f670f959e224d5f0ad58a642e8c...
:sunny: Test successful - checks-actions Approved by: nrc Pushing 4b95a894b8395f670f959e224d5f0ad58a642e8c to master...
The first commit is some minor refactoring. The second commit does the 'obvious' work of making DispatchFromDyn well-known. In order to properly enforce the wf-ness constraints on DispatchFromDyn, Chalk needs to know about one-zst types, that is in the next commit. The final commit finishes the DispatchFromDyn work by enforcing the WF-ness constraints and adding tests.
r? @jackh726