don't overwrite completion_time

spandex-project / spandex

A platform agnostic tracing library

MIT License

335 stars 53 forks source link

don't overwrite completion_time #61

Closed zachdaniel closed 6 years ago

sourcelevel-bot[bot] commented 6 years ago

Ebert has finished reviewing this Pull Request and has found:

2 possible new issues (including those that may have been commented here).

You can see more details about this review at https://ebertapp.io/github/zachdaniel/spandex/pulls/61.

coveralls commented 6 years ago

Coverage increased (+2.04%) to 72.683% when pulling c27345195ad860cfad7b9a14743fe1e49e005047 on fix-time-overwriting into b8779e1b157cc38f5df9fcfaf9f773b3537a1e4a on master.