#33140: order of checks
-----------------------------------------+------------------------
               Reporter:  David Szotten  |          Owner:  nobody
                   Type:  Uncategorized  |         Status:  new
              Component:  Uncategorized  |        Version:  3.2
               Severity:  Normal         |       Keywords:
           Triage Stage:  Unreviewed     |      Has patch:  0
    Needs documentation:  0              |    Needs tests:  0
Patch needs improvement:  0              |  Easy pickings:  0
                  UI/UX:  0              |
-----------------------------------------+------------------------
 https://github.com/django/django/pull/8967 changes the checks registry to
 use sets instead of lists. i may be missing it, but i can't see a ticket
 reference or any reason for the change

 i just had a bug where the behaviour of a generated file (happened to be
 by https://github.com/adenh93/django-typomatic) depends on collecting
 classes, which in turn depends on the import order of files in my project

 i had two different invocations get different orderings of checks causing
 different import orders

 could we consider switching back to lists? or e.g. using `sorted` instead
 of `list` in `get_checks`
 
https://github.com/django/django/blob/8036b53de61f16098a2f22c50621987182bdfaed/django/core/checks/registry.py#L96

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

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/055.2953a9b5a4a8f39fdfe6a4c843f7f9c1%40djangoproject.com.

Reply via email to