Django 1.9 app_label #420

Merged
merged 1 commit into from Jun 23, 2016

Projects

None yet

4 participants

@jpwhite3
Contributor

Adding app_label to Tag and TaggedItems models for Django 1.9 compatibility. Without specifying an app_label you MUST add taggit to INSTALLED_APPS, which you may not want to do if you plan to customize the models.
#330

@jpwhite3 jpwhite3 Update models.py
Adding app_label to models for Django 1.9 compatibility.
2a86ccd
@codecov-io
codecov-io commented Jun 22, 2016 edited

Current coverage is 90.55%

Merging #420 into master will decrease coverage by 0.11%

@@             master       #420   diff @@
==========================================
  Files             8          8          
  Lines           697        699     +2   
  Methods           0          0          
  Messages          0          0          
  Branches        114        114          
==========================================
+ Hits            632        633     +1   
  Misses           53         53          
- Partials         12         13     +1   

Powered by Codecov. Last updated by 4027dfd...2a86ccd

@jpwhite3 jpwhite3 changed the title from Update models.py to Django 1.9 app_label Jun 22, 2016
@prydie
Contributor
prydie commented Jun 23, 2016

Looks good to me. Thanks for your submission.

@frewsxcv, you happy to merge?

@frewsxcv
Collaborator

Thanks!

@frewsxcv frewsxcv merged commit fb29674 into alex:master Jun 23, 2016

3 checks passed

codecov/patch 100% of diff hit (target 90.67%)
Details
codecov/project Absolute coverage decreased by -0.11% but relative coverage increased by +9.32% compared to 4027dfd
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment