search

janet-lang / janet-lang.org

Website for janet
https://janet-lang.org
MIT License
92 stars 58 forks source link

Update supervisor event description #180

Closed sogaiu closed 1 year ago

sogaiu commented 1 year ago

In my local testing, I'm observing supervisor events that are tuples with 3 elements, though the current website docs describe such events as being tuples of two elements. Perhaps things changed at some point (or may be I'm doing something wrong (^^; )

Also, the keyword that's the first element of the tuple seems to currently be :ok and not :done for the non-error case.

This PR is an attempt to reflect these findings in the docs.

I've tried to describe the third element of the tuple based on code from here. Happy to change to a better description :)

bakpakin commented 1 year ago

LGTM - yes the task-id is the new element to make it easier to distinguish where an event came from without hacking the payload (in the threaded case specifically).