llazzaro / django-scheduler / 550

85%

Pull #313

travis-ci

Remove null from all model fields of type CharField and TextField

Django recommends against using null=True on string-based fields. This
change brings the models in line with Django conventions and suggested
practices. This avoids two different values to represent "empty".

See Django docs for more details:

https://docs.djangoproject.com/en/1.11/ref/models/fields/#null

> Avoid using null on string-based fields such as CharField and
> TextField. If a string-based field has null=True, that means it has
> two possible values for “no data”: NULL, and the empty string. In most
> cases, it’s redundant to have two possible values for “no data;” the
> Django convention is to use the empty string, not NULL.

17 of 17 new or added lines in 3 files covered. (100.0%)

1236 of 1806 relevant lines covered (68.44%)

7.51 hits per line