[Django] #34980: Changed migration operation dependencies to namedtuples.

5 views
Skip to first unread message

Django

unread,
Nov 20, 2023, 5:00:29 AM11/20/23
to django-...@googlegroups.com
#34980: Changed migration operation dependencies to namedtuples.
-------------------------------------+-------------------------------------
Reporter: Mariusz | Owner: Mariusz Felisiak
Felisiak |
Type: | Status: assigned
Cleanup/optimization |
Component: | Version: dev
Migrations |
Severity: Normal | Keywords:
Triage Stage: | Has patch: 0
Unreviewed |
Needs documentation: 0 | Needs tests: 0
Patch needs improvement: 0 | Easy pickings: 0
UI/UX: 0 |
-------------------------------------+-------------------------------------
Using `namedtuple`s and `Enum`s should make the migration autodetector
more readable and maintainable.

--
Ticket URL: <https://code.djangoproject.com/ticket/34980>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

Django

unread,
Nov 20, 2023, 5:01:14 AM11/20/23
to django-...@googlegroups.com
#34980: Changed migration operation dependencies to namedtuples.
-------------------------------------+-------------------------------------
Reporter: Mariusz Felisiak | Owner: Mariusz
Type: | Felisiak
Cleanup/optimization | Status: assigned
Component: Migrations | Version: dev
Severity: Normal | Resolution:
Keywords: | Triage Stage:
| Unreviewed
Has patch: 1 | Needs documentation: 0

Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by Mariusz Felisiak):

* has_patch: 0 => 1


Comment:

I've been thinking about changing this for some time, and I'm
[https://github.com/django/django/pull/16905#discussion_r1306500610 not
the only one], so I gave it a try.

[https://github.com/django/django/pull/17494 PR]

--
Ticket URL: <https://code.djangoproject.com/ticket/34980#comment:1>

Reply all
Reply to author
Forward
0 new messages