graphite-project / graphite-web

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

Preparing master to Django 3.2 #2766

Closed deniszh closed 1 year ago

deniszh commented 2 years ago

Removing python 2.7 from setup files and tests, testing new python and Django only

codecov-commenter commented 2 years ago

Codecov Report

Base: 76.30% // Head: 76.28% // Decreases project coverage by -0.01% :warning:

Coverage data is based on head (89a69f3) compared to base (b52987a). Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #2766 +/- ## ========================================== - Coverage 76.30% 76.28% -0.02% ========================================== Files 88 88 Lines 9622 9623 +1 Branches 2154 2154 ========================================== - Hits 7342 7341 -1 - Misses 2014 2015 +1 - Partials 266 267 +1 ``` | [Impacted Files](https://codecov.io/gh/graphite-project/graphite-web/pull/2766?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=graphite-project) | Coverage Δ | | |---|---|---| | [webapp/graphite/app\_settings.py](https://codecov.io/gh/graphite-project/graphite-web/pull/2766/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=graphite-project#diff-d2ViYXBwL2dyYXBoaXRlL2FwcF9zZXR0aW5ncy5weQ==) | `70.96% <ø> (ø)` | | | [webapp/graphite/render/glyph.py](https://codecov.io/gh/graphite-project/graphite-web/pull/2766/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=graphite-project#diff-d2ViYXBwL2dyYXBoaXRlL3JlbmRlci9nbHlwaC5weQ==) | `41.19% <100.00%> (ø)` | | | [webapp/graphite/settings.py](https://codecov.io/gh/graphite-project/graphite-web/pull/2766/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=graphite-project#diff-d2ViYXBwL2dyYXBoaXRlL3NldHRpbmdzLnB5) | `75.59% <100.00%> (+0.11%)` | :arrow_up: | | [webapp/graphite/tags/redis.py](https://codecov.io/gh/graphite-project/graphite-web/pull/2766/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=graphite-project#diff-d2ViYXBwL2dyYXBoaXRlL3RhZ3MvcmVkaXMucHk=) | `85.80% <0.00%> (-1.24%)` | :arrow_down: | Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=graphite-project). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=graphite-project)

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

lgtm-com[bot] commented 1 year ago

This pull request fixes 2 alerts when merging 89a69f30cd215aa24a0fdcde9c6d52e07b1616b3 into ed08fa4e2f6526ae11e55f88f70d0b6d36e4052f - view on LGTM.com

fixed alerts: