SolarEdgeTech / pyctuator

Monitor Python applications using Spring Boot Admin
Apache License 2.0
175 stars 18 forks source link

Upgrade flask to ^2.3.0 #99

Closed michaelyaakoby closed 1 year ago

michaelyaakoby commented 1 year ago

This required chaning the patching of jsonifying of datetime/date values to use ISO format.

As of 2.2.*, changing the datetime and date JSON encoding is done globally, see https://stackoverflow.com/a/74618781/2692895 (which is an updated reply to https://stackoverflow.com/questions/43663552/keep-a-datetime-date-in-yyyy-mm-dd-format-when-using-flasks-jsonify)

codecov-commenter commented 1 year ago

Codecov Report

Patch coverage: 100.00% and project coverage change: +0.05% :tada:

Comparison is base (dffb45c) 95.18% compared to head (7d2902c) 95.23%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #99 +/- ## ========================================== + Coverage 95.18% 95.23% +0.05% ========================================== Files 28 28 Lines 1163 1155 -8 ========================================== - Hits 1107 1100 -7 + Misses 56 55 -1 ``` | [Files Changed](https://app.codecov.io/gh/SolarEdgeTech/pyctuator/pull/99?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None) | Coverage Δ | | |---|---|---| | [pyctuator/impl/fastapi\_pyctuator.py](https://app.codecov.io/gh/SolarEdgeTech/pyctuator/pull/99?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None#diff-cHljdHVhdG9yL2ltcGwvZmFzdGFwaV9weWN0dWF0b3IucHk=) | `99.01% <100.00%> (-0.03%)` | :arrow_down: | | [pyctuator/impl/flask\_pyctuator.py](https://app.codecov.io/gh/SolarEdgeTech/pyctuator/pull/99?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None#diff-cHljdHVhdG9yL2ltcGwvZmxhc2tfcHljdHVhdG9yLnB5) | `100.00% <100.00%> (+1.02%)` | :arrow_up: |

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

michaelyaakoby commented 1 year ago

Fixes #97