#20653 closed Cleanup/optimization (fixed)
Rename new checksetup manage command
Reported by: | Russell Keith-Magee | Owned by: | nobody |
---|---|---|---|
Component: | Core (Management commands) | Version: | dev |
Severity: | Release blocker | Keywords: | |
Cc: | Triage Stage: | Accepted | |
Has patch: | no | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description (last modified by )
In commit 91f317c7 Daniel committed a new management command called "checksetup" to perform warning-level checks for problems that may occur on upgrading between versions.
There is some overlap here between this new feature, and the scope of the 2013 GSoC project to refactor the "validate" command. One of the aims of this project is to reduce the naming collisions around "validate" -- it could refer to "form validation", "model validation" or "field validation", depending on how it's used.
This GSoC project will also involve the introduction of a new management command as part of reducing this naming collision; one of the candidate names was "check"
Before 1.6 is finalized, we should rename checksetup so it can be used as a generic command for all checks, rather than just the single-purpose check it's currently performing.
Change History (7)
comment:1 by , 11 years ago
comment:2 by , 11 years ago
Description: | modified (diff) |
---|
comment:3 by , 11 years ago
Consensus on mailing list seems to be that "check" is the best name; I've got a commit for this that will land shortly.
comment:4 by , 11 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
Trac triggers seem to be busted; this was committed in 03465639
comment:5 by , 11 years ago
Resolution: | fixed |
---|---|
Status: | closed → new |
checksetup is still in the django-admin docs
https://github.com/django/django/blob/master/docs/ref/django-admin.txt#L92
https://github.com/django/django/pull/1353
(i figure this is small enough to not warrant a new ticket number.)
comment:6 by , 11 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
Mailing list discussion here