Andrew Dunn
Andrew Dunn
Issue discussion here: #156
Can do! Don't review this until I've done that, I've found some more avenues for refactoring.
@bramp Good to go! Also, I added an error message for the issue described in the main pull-request post.
``` participant Actor note right of Actor: This actor does not currently have an execution in progress, so level = -1 Actor->+Actor: Increase level note right of Actor: An execution...
Made some changes addressing your comments.
Hey, @bramp just a friendly ping to see the progress of this review? Cheers!
I'm currently working on an implementation of this using my proposed syntax. I _may_ do a simpler enhancement to learn the codebase before attempting it though. (probably won't)
Progress so far, the following code generates the given sequence diagram: ``` Bob->+Alice: Say "Hello". +Alice->-Alice: Process salutation. -Alice->+Bob: Say "Goodbye". note right of Bob: Reflect on life ``` **TODO**...
Unit/regression tests written. All that's left is to fix this rendering bug related to self signals that both end and begin an ExecutionSpecification, as well as bringing the code up...