Opened 9 years ago

Closed 9 years ago

#25350 closed Cleanup/optimization (fixed)

Add alias '--no-input' to '--noinput' options for naming consistency

Reported by: David Sanders Owned by: Raphael Michel
Component: Core (Management commands) Version: dev
Severity: Normal Keywords: 1.9
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

I've been caught out a few times with the various management commands that use --noinput, by specifying --no-input in a similar naming fashion to other options.

Was wondering whether it's worth adding an alias to existing --noinput options?

Change History (7)

comment:1 by Markus Holtermann, 9 years ago

Keywords: 1.9 added
Triage Stage: Unreviewed → Accepted
Version: 1.8 → master

Would be nice to get this into 1.9.

comment:2 by David Sanders, 9 years ago

Possible options:

Separate option:

  --no-input            Do NOT prompt the user for input of any kind.
  --noinput             (Deprecated) Alias for --no-input.

or as an alias:

  --no-input, --noinput
                        Do NOT prompt the user for input of any kind.

comment:3 by Tim Graham, 9 years ago

I think the alias option is fine. I doubt removing the old option at some point will be worthwhile.

comment:4 by Raphael Michel, 9 years ago

Owner: changed from nobody to Raphael Michel
Status: new → assigned

comment:5 by Raphael Michel, 9 years ago

Has patch: set

I created a pull request: ​https://github.com/django/django/pull/5242

comment:6 by Tim Graham, 9 years ago

Triage Stage: Accepted → Ready for checkin

comment:7 by Tim Graham <timograham@…>, 9 years ago

Resolution: → fixed
Status: assigned → closed

In 1bbca79:

Fixed #25350 -- Added alias --no-input for --noinput to management commands.

Note: See TracTickets for help on using tickets.
Back to Top