Opened 15 years ago

Last modified 14 years ago

#12221 new New feature

Allow custom management commands to define bash completion arguments

Reported by: Eric Holscher Owned by: nobody
Component: Core (Management commands) Version: 1.1
Severity: Normal Keywords:
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: yes
Needs tests: no Patch needs improvement: yes
Easy pickings: no UI/UX: no
Pull Requests:How to create a pull request

Description

What it says on the tin. Sent a proposal to -dev, this ticket is to track the progress of the patches.

http://groups.google.com/group/django-developers/browse_thread/thread/a076744059cace3f

According to the ticket's flags, the next step(s) to move this issue forward are:

  • To write documentation for the patch, then uncheck "Needs documentation" on the ticket.
  • To improve the patch as described in the pull request review comments or on this ticket, then uncheck "Patch needs improvement".
  • If creating a new pull request, include a link to the pull request in the ticket comment when making that update. The usual format is: [https://github.com/django/django/pull/#### PR].

Change History (10)

by Eric Holscher, 15 years ago

Attachment: completion_tests.diff added

Tests for current bash completion.

by Eric Holscher, 15 years ago

Attachment: full_completion.diff added

All of the work I've done thus far for this ticket.

comment:1 by Eric Holscher, 15 years ago

Has patch: set
Needs documentation: set
Patch needs improvement: set

This would be nice to commit along with #9170, which gives some-what real docs for management commands. I will add docs to this patch that add to the ones on those, if it gets accepted.

comment:2 by Chris Beaven, 15 years ago

Note: I've got a working branch that extends the current behavior to show apps for any management commands which subclasses AppCommand

comment:3 by Eric Holscher, 15 years ago

Hey Chris,

Thinking about trying to get a version of this into 1.2, if you could share your patch somewhere (here, or a pointer to github), that would be awesome. Thanks.

comment:4 by Chris Beaven, 15 years ago

I got it working, but I can't vouch for whether it's all best practice - was just a bit of hacking for me: http://github.com/SmileyChris/django/commits/bash

comment:5 by Russell Keith-Magee, 15 years ago

Triage Stage: UnreviewedAccepted

comment:6 by Matt McClanahan, 14 years ago

Component: UncategorizedCore (Management commands)
Severity: Normal
Type: New feature

comment:7 by Aymeric Augustin, 13 years ago

UI/UX: unset

Change UI/UX from NULL to False.

comment:8 by Aymeric Augustin, 13 years ago

Easy pickings: unset

Change Easy pickings from NULL to False.

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