The-Academic-Observatory / observatory-platform

Observatory Platform Package
https://docs.observatory.academy
Apache License 2.0
18 stars 5 forks source link

Added timestamp to json parse #635

Closed keegansmith21 closed 1 year ago

keegansmith21 commented 1 year ago

Updated the load_and_parse_json() function to parse timestamps separately to dates. Also added custom parsing fields.

Also added some tests for the function.

codecov[bot] commented 1 year ago

Codecov Report

Patch coverage: 89.18% and project coverage change: +0.41% :tada:

Comparison is base (8aa0338) 86.89% compared to head (46af980) 87.30%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop #635 +/- ## =========================================== + Coverage 86.89% 87.30% +0.41% =========================================== Files 34 34 Lines 4654 4680 +26 Branches 563 572 +9 =========================================== + Hits 4044 4086 +42 + Misses 495 477 -18 - Partials 115 117 +2 ``` | [Files Changed](https://app.codecov.io/gh/The-Academic-Observatory/observatory-platform/pull/635?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The-Academic-Observatory) | Coverage Δ | | |---|---|---| | [...rm/observatory/platform/observatory\_environment.py](https://app.codecov.io/gh/The-Academic-Observatory/observatory-platform/pull/635?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The-Academic-Observatory#diff-b2JzZXJ2YXRvcnktcGxhdGZvcm0vb2JzZXJ2YXRvcnkvcGxhdGZvcm0vb2JzZXJ2YXRvcnlfZW52aXJvbm1lbnQucHk=) | `85.48% <89.18%> (+3.62%)` | :arrow_up: |

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.