refactor: use CodeBuilder#189
Conversation
|
Just a quick aside - but I'd recommend switching over to using projects (i.e. defined by a I'm hoping to expand |
|
+1 on landing this as the official guidance. I did a very similar migration on #192 last week while fixing tutorial issues from #191 and converged on the same shape, so independent confirmation it's the right move. @bitwalker I'm bought on project-assembler as the destination, just not for tutorials in this iteration. Couldn't find any Happy to take a fresh PR at it once Should we open an issue for this? |
I think #187 got closed when the base branch got merged.
Official guidance should be to utilize the
CodeBuilder, which is also provided byClient. Recently mixing source managers in assemblers started resulting in internal panics (0xMiden/miden-vm#2778), so we need to make guidance as consistent as possible. This also simplifies code a bunch. cc @Keinberger @BrianSeong99Closes #188.