Skip to content

Conversation

David-Wobrock
Copy link
Collaborator

Fixes #138
Fixes #169

@David-Wobrock David-Wobrock force-pushed the handle-unknown-db-vendor branch from 47466c9 to ad189b0 Compare January 28, 2022 21:11
@codecov-commenter
Copy link

codecov-commenter commented Jan 28, 2022

Codecov Report

Merging #195 (ad189b0) into main (71ddadd) will decrease coverage by 0.18%.
The diff coverage is 73.68%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #195      +/-   ##
==========================================
- Coverage   93.68%   93.50%   -0.19%     
==========================================
  Files          79       79              
  Lines        1758     1770      +12     
==========================================
+ Hits         1647     1655       +8     
- Misses        111      115       +4     
Impacted Files Coverage Δ
django_migration_linter/sql_analyser/analyser.py 80.76% <61.53%> (-13.35%) ⬇️
...ation_linter/management/commands/lintmigrations.py 89.02% <100.00%> (+0.27%) ⬆️
django_migration_linter/migration_linter.py 84.38% <100.00%> (+0.04%) ⬆️
django_migration_linter/sql_analyser/__init__.py 100.00% <100.00%> (ø)
tests/unit/test_sql_analyser.py 100.00% <100.00%> (ø)

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 71ddadd...ad189b0. Read the comment docs.

@David-Wobrock David-Wobrock merged commit 76a55db into main Jan 30, 2022
@David-Wobrock David-Wobrock deleted the handle-unknown-db-vendor branch January 30, 2022 14:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

PostGis as database_vendor Linter should fail, or at least warn, when ran against an unsupported vendor
2 participants