Do not merge [issue #110] Wireformat checks property as object, not array, keyed…

[xstefank]

… by check name

issue #110

[antoinesd]

LGTM

[Emily-Jiang]

I am not sure whether this makes sense any more. I tried and found out health check names do not need to be unique.

0
data
stop	true
name	"serviceA"
state	"UP"
1
data
lifetime	365
name	"serviceA"
state	"UP"
outcome	"UP"

[Emily-Jiang]

I suggest not to do this change as I don't see much benefit to introduce this change at all but cause more issues.

[xstefank]

@Emily-Jiang just for completeness - the unique names have been discussed in #34

[Emily-Jiang]

I have added a comment on that issue. Unless there is a business reason, I prefer not to do this as it will break old apps for no strong reason.

[antoinesd]

@Emily-Jiang one of the idea is to enforce uniqueness of check's name and to align on other practice around. Like it was discussed in #110, it is still possible to keep the old format for backward compatibility (with a flag for instance)

[Emily-Jiang]

I don't see any real benefit brought by this change except introducing breaking changes.

[xstefank]

Closing this PR. We will try this feature in the implementations first.