Closed paulfalgout closed 5 years ago
We're missing coverage now here https://github.com/marionettejs/backbone.marionette/blob/master/src/common/monitor-view-events.js#L30
I believe the coverage loss likely occurred here #3636
Because there are no more destroys during destroys it may have rendered this check unnecessary. But we should be absolutely sure there is not a corner case before removing this line.
cc: @taburetkin
We're missing coverage now here https://github.com/marionettejs/backbone.marionette/blob/master/src/common/monitor-view-events.js#L30
I believe the coverage loss likely occurred here #3636
Because there are no more destroys during destroys it may have rendered this check unnecessary. But we should be absolutely sure there is not a corner case before removing this line.
cc: @taburetkin