3YOURMIND / django-migration-linter

:rocket: Detect backward incompatible migrations for your django project
https://pypi.python.org/pypi/django-migration-linter/
Apache License 2.0
530 stars 58 forks source link

Add internal classes and add type hints + mypy. #244

Closed David-Wobrock closed 1 year ago

David-Wobrock commented 1 year ago

Fixes https://github.com/3YOURMIND/django-migration-linter/issues/238

codecov-commenter commented 1 year ago

Codecov Report

Base: 93.73% // Head: 93.94% // Increases project coverage by +0.20% :tada:

Coverage data is based on head (9669cd2) compared to base (847ac12). Patch coverage: 96.91% of modified lines in pull request are covered.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #244 +/- ## ========================================== + Coverage 93.73% 93.94% +0.20% ========================================== Files 79 77 -2 Lines 1853 1966 +113 ========================================== + Hits 1737 1847 +110 - Misses 116 119 +3 ``` | [Impacted Files](https://codecov.io/gh/3YOURMIND/django-migration-linter/pull/244?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None) | Coverage Δ | | |---|---|---| | [django\_migration\_linter/utils.py](https://codecov.io/gh/3YOURMIND/django-migration-linter/pull/244/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None#diff-ZGphbmdvX21pZ3JhdGlvbl9saW50ZXIvdXRpbHMucHk=) | `92.10% <77.77%> (-4.96%)` | :arrow_down: | | [django\_migration\_linter/sql\_analyser/analyser.py](https://codecov.io/gh/3YOURMIND/django-migration-linter/pull/244/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None#diff-ZGphbmdvX21pZ3JhdGlvbl9saW50ZXIvc3FsX2FuYWx5c2VyL2FuYWx5c2VyLnB5) | `93.33% <87.50%> (-2.83%)` | :arrow_down: | | [django\_migration\_linter/migration\_linter.py](https://codecov.io/gh/3YOURMIND/django-migration-linter/pull/244/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None#diff-ZGphbmdvX21pZ3JhdGlvbl9saW50ZXIvbWlncmF0aW9uX2xpbnRlci5weQ==) | `85.15% <90.00%> (+0.15%)` | :arrow_up: | | [django\_migration\_linter/\_\_init\_\_.py](https://codecov.io/gh/3YOURMIND/django-migration-linter/pull/244/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None#diff-ZGphbmdvX21pZ3JhdGlvbl9saW50ZXIvX19pbml0X18ucHk=) | `100.00% <100.00%> (ø)` | | | [django\_migration\_linter/cache.py](https://codecov.io/gh/3YOURMIND/django-migration-linter/pull/244/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None#diff-ZGphbmdvX21pZ3JhdGlvbl9saW50ZXIvY2FjaGUucHk=) | `100.00% <100.00%> (ø)` | | | [django\_migration\_linter/constants.py](https://codecov.io/gh/3YOURMIND/django-migration-linter/pull/244/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None#diff-ZGphbmdvX21pZ3JhdGlvbl9saW50ZXIvY29uc3RhbnRzLnB5) | `100.00% <100.00%> (ø)` | | | [...ation\_linter/management/commands/lintmigrations.py](https://codecov.io/gh/3YOURMIND/django-migration-linter/pull/244/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None#diff-ZGphbmdvX21pZ3JhdGlvbl9saW50ZXIvbWFuYWdlbWVudC9jb21tYW5kcy9saW50bWlncmF0aW9ucy5weQ==) | `93.10% <100.00%> (+0.16%)` | :arrow_up: | | [...ation\_linter/management/commands/makemigrations.py](https://codecov.io/gh/3YOURMIND/django-migration-linter/pull/244/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None#diff-ZGphbmdvX21pZ3JhdGlvbl9saW50ZXIvbWFuYWdlbWVudC9jb21tYW5kcy9tYWtlbWlncmF0aW9ucy5weQ==) | `96.61% <100.00%> (+0.18%)` | :arrow_up: | | [django\_migration\_linter/management/utils.py](https://codecov.io/gh/3YOURMIND/django-migration-linter/pull/244/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None#diff-ZGphbmdvX21pZ3JhdGlvbl9saW50ZXIvbWFuYWdlbWVudC91dGlscy5weQ==) | `82.60% <100.00%> (+1.65%)` | :arrow_up: | | [django\_migration\_linter/operations.py](https://codecov.io/gh/3YOURMIND/django-migration-linter/pull/244/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None#diff-ZGphbmdvX21pZ3JhdGlvbl9saW50ZXIvb3BlcmF0aW9ucy5weQ==) | `85.71% <100.00%> (+1.09%)` | :arrow_up: | | ... and [67 more](https://codecov.io/gh/3YOURMIND/django-migration-linter/pull/244/diff?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None) | | 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=None). 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=None)

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