Using `Option<NonEmptyChunk>` for `pendingEvents` is more precise than `Schema.Chunk`

pendingEvents

pendingEvents

pendingEvents

pendingEvents should never empty, so

Option<NonEmptyChunk>

Option<NonEmptyChunk>

Option<NonEmptyChunk>

Option<NonEmptyChunk> is more semantically precise:

None

None

None

None means no pending events,

Some

Some

Some

Some means pending events > 0

 pendingEvents: Schema.optional(Schema.NonEmptyChunk(Event.EventSchema)),

 pendingEvents: Schema.optional(Schema.NonEmptyChunk(Event.EventSchema)),

 pendingEvents: Schema.optional(Schema.NonEmptyChunk(Event.EventSchema)),

 pendingEvents: Schema.optional(Schema.NonEmptyChunk(Event.EventSchema)),

pendingEvents: Schema.Chunk...

pendingEvents: Schema.Chunk...

pendingEvents: Schema.Chunk...

pendingEvents: Schema.Chunk...

agreed ?

No replies yetBe the first to reply to this messageJoin