#16247 closed Cleanup/optimization (fixed)
Remove the wildcard feature of INSTALLED_APPS
Reported by: | Aymeric Augustin | Owned by: | nobody |
---|---|---|---|
Component: | Core (Other) | Version: | dev |
Severity: | Normal | Keywords: | |
Cc: | Triage Stage: | Design decision needed | |
Has patch: | yes | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description
As discussed on django-developers.
Attachments (1)
Change History (10)
comment:1 by , 13 years ago
Owner: | changed from | to
---|
by , 13 years ago
Attachment: | 16247.diff added |
---|
comment:2 by , 13 years ago
Has patch: | set |
---|
comment:3 by , 13 years ago
Triage Stage: | Unreviewed → Accepted |
---|
follow-up: 5 comment:4 by , 13 years ago
Triage Stage: | Accepted → Design decision needed |
---|
follow-up: 6 comment:5 by , 13 years ago
Replying to jezdez:
As mentioned in the thread on http://groups.google.com/group/django-developers/browse_thread/thread/b35325a078f7a912 this needs a bit of feedback from someone that was involved in adding this feature at the time.
I don't see that requirement ("needs feedback from someone involved in adding the feature") mentioned in that thread anywhere? I said "if someone pops up" -- I was going to just wait a couple days to see if anyone objects in that thread, and if not, commit this. Personally I don't feel any need to get review from someone involved in adding the feature.
comment:6 by , 13 years ago
Replying to carljm:
Replying to jezdez:
As mentioned in the thread on http://groups.google.com/group/django-developers/browse_thread/thread/b35325a078f7a912 this needs a bit of feedback from someone that was involved in adding this feature at the time.
I don't see that requirement ("needs feedback from someone involved in adding the feature") mentioned in that thread anywhere? I said "if someone pops up" -- I was going to just wait a couple days to see if anyone objects in that thread, and if not, commit this. Personally I don't feel any need to get review from someone involved in adding the feature.
In that case I must have misunderstood your estimation of time when you wrote "if someone pops up". It wrongly indicated to me that you want to genuinely wait for someone that introduced that feature a while ago to be able to explain or even speak against removing of that feature. Without that type of response on the mailing list I find it rather hard to decide for or against the removal of a feature -- whether documented or not. Gladly, it's a minor feature that falls in the category of "nitpicking" so can easily be handled at a later time in a DDN session.
comment:7 by , 13 years ago
Owner: | changed from | to
---|
The patch is easy to recreate if it breaks — it just removes a paragraph of code and adds a paragraph of text.
As mentioned in the thread on http://groups.google.com/group/django-developers/browse_thread/thread/b35325a078f7a912 this needs a bit of feedback from someone that was involved in adding this feature at the time.