Conversation
|
I clicked Approve because this indeed fixed all the examples shown above. However, there was a yet another variation I floated yesterday to @nwt that I now see is still divergent here from how ZSON shows it.
This branch at commit 3726e3d still shows an explicit decorator here. I discussed this with @nwt and he pointed out that having an explicit decorator in this case is not wrong necessarily, since the Type Decorators part of the ZSON spec points out where they "might be required" with unions to remove ambiguity, but that does not preclude an implementation from choosing to be explicit and include them even if there's no ambiguity. That said, I'm still not sure if there's a problem here. It's not like I can cut & paste the output as-is and treat it as ZSON since there's additional decoration unique to the app (e.g., the array indexes like It looks like it's showing a union inside of a union. @nwt: Is that something that should be addressed? |

That empties file now looks like this:
Closes #2454