Opened 9 years ago

Closed 9 years ago

#24514 closed Cleanup/optimization (fixed)

Unused import on some auto generated migrations

Reported by: Hugo Tácito Owned by: Chris Luc
Component: Migrations Version: 1.7
Severity: Normal Keywords: unused import migrations
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no


Some auto generated migrations don't use django.db.models. Especially those that only change the Meta class. It would be nice to change the django.db.migrations.writer.MIGRATION_TEMPLATE to only add the from django.db import models if its really needed.

Example of auto generated migration with unused import:

# -*- coding: utf-8 -*-
from __future__ import unicode_literals

from django.db import models, migrations

class Migration(migrations.Migration):

    dependencies = [
        ('city', '0003_auto_20150305_0928'),

    operations = [
            options={'verbose_name': 'City', 'verbose_name_plural': 'Cities'},

Change History (7)

comment:1 by Tim Graham, 9 years ago

Triage Stage: UnreviewedAccepted

comment:2 by Chris Luc, 9 years ago

Owner: changed from nobody to Chris Luc
Status: newassigned

comment:3 by Markus Holtermann, 9 years ago

I think models is a convenience import and should stay around.

comment:4 by Chris Luc, 9 years ago

I think having models in a migration that doesn't use the import might spark confusion for why it's being imported in the first place. I think it makes the migrations cleaner by not having unused imports. I don't see many situations where someone would manually need the models import. But, in the rare occasion that they do need it, manually adding it is not that bad. @MarkusH

comment:5 by Hugo Tácito, 9 years ago

Personally I'm having problems when I do automated code analysis with pylint. I know this can be solved just ignoring the migrations folder but it would be nice if django only do this import when necessary on auto generated migrations.

comment:6 by Chris Luc, 9 years ago

Has patch: set

comment:7 by Tim Graham <timograham@…>, 9 years ago

Resolution: fixed
Status: assignedclosed

In a7bc00e1:

Fixed #24514 -- Made migration writer omit models import if it's unused.

Note: See TracTickets for help on using tickets.
Back to Top