graphite-project / graphite-web

A highly scalable real-time graphing system
http://graphite.readthedocs.org/
Apache License 2.0
5.89k stars 1.26k forks source link

Forward port changes from #2729 to master #2730

Closed deniszh closed 2 years ago

deniszh commented 2 years ago

Forward port changes from #2729 to master

Deepsource Errors coming mostly from umsgpack and old tests.

codecov-commenter commented 2 years ago

Codecov Report

Merging #2730 (eb831cb) into master (5f35fc2) will decrease coverage by 0.85%. The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2730      +/-   ##
==========================================
- Coverage   77.09%   76.23%   -0.86%     
==========================================
  Files         176       88      -88     
  Lines       18972     9593    -9379     
  Branches     4034     2149    -1885     
==========================================
- Hits        14626     7313    -7313     
+ Misses       3814     2014    -1800     
+ Partials      532      266     -266     
Impacted Files Coverage Δ
webapp/graphite/umsgpack.py 0.00% <0.00%> (ø)
graphite/events/admin.py
graphite/functions/views.py
graphite/intervals.py
graphite/url_shortener/baseconv.py
graphite/tags/localdatabase.py
graphite/events/migrations/0001_initial.py
graphite/dashboard/send_graph.py
graphite/umsgpack.py
graphite/finders/ceres.py
... and 79 more

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 523c576...eb831cb. Read the comment docs.

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert when merging eb831cbb909d6973d1cf53453615b206f3aa8415 into 523c57609f7354617696efafe84d1833b1620c6b - view on LGTM.com

new alerts: