Opened 10 years ago

Last modified 11 months ago

#24901 new Cleanup/optimization

makemigrations should create empty migrations dir for any installed app without it

Reported by: Andriy Sokolovskiy Owned by: nobody
Component: Migrations Version: dev
Severity: Normal Keywords:
Cc: Ülgen Sarıkavak Triage Stage: Someday/Maybe
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no
Pull Requests:How to create a pull request

Description

Since django is not supporting apps without migrations in master, makemigrations should create empty migrations dir for any installed app without it, because it is ignoring applications if one use manage.py makemigrations without explicit providing of application name.

According to the ticket's flags, the next step(s) to move this issue forward are:

  • Unknown. The Someday/Maybe triage stage is used to keep track of high-level ideas or long term feature requests.

    It could be an issue that's blocked until a future version of Django (if so, Keywords will contain that version number). It could also be an enhancement request that we might consider adding someday to the framework if an excellent patch is submitted.

    If you're interested in contributing to the issue, raising your ideas on the Django Forum would be a great place to start.

Change History (6)

comment:1 by Tim Graham, 10 years ago

Triage Stage: UnreviewedAccepted

But don't create a migrations directory unless the app has models.

comment:2 by Markus Holtermann, 10 years ago

We haven't really deprecated apps without migrations (no deprecation warning in either 1.7 or 1.8). Although we have documented it I'm not sure we can make the change for 1.9.

We also need to figure out a way of how to run the tests without explicit migrations. Once we're there I think we're ready for apps without migrations.

comment:3 by Markus Holtermann, 10 years ago

After a talk with Andrew at DjangoCon Europe:

Given #24481 and how that plays in with our test apps, we might reconsider dropping support for apps without migrations.

At some point (2.0 would be a good candidate to justify increasing the major version number ;) ) we should automatically create the migrations package.

comment:4 by Andriy Sokolovskiy, 10 years ago

After discussion with Markus, we should postpone this until the result of #24481

comment:5 by Claude Paroz, 10 years ago

Triage Stage: AcceptedSomeday/Maybe

comment:6 by Ülgen Sarıkavak, 11 months ago

Cc: Ülgen Sarıkavak added
Note: See TracTickets for help on using tickets.
Back to Top