Closed andrea-pasquale closed 3 months ago
Attention: Patch coverage is 97.61905%
with 2 lines
in your changes missing coverage. Please review.
Project coverage is 97.24%. Comparing base (
41439c7
) to head (d2cae97
). Report is 92 commits behind head on main.
In this PR I address some of the comments raised in #865, given that the changes were substantial I decided to open a separate PR.
History
load
from ExecutorThere might be a better way to (de)serialize
History
, the current implementation should be acceptable. The only annoying object left to deal with seems to bemeta.json
, we could think about a way of moving its content somewhere else.