#14187: Add support for bpython in django-admin.py shell
--------------------------------------+-------------------------------------
          Reporter:  jdunck           |         Owner:  nobody
            Status:  new              |     Milestone:  1.3   
         Component:  django-admin.py  |       Version:  1.2   
        Resolution:                   |      Keywords:        
             Stage:  Accepted         |     Has_patch:  1     
        Needs_docs:  0                |   Needs_tests:  1     
Needs_better_patch:  0                |  
--------------------------------------+-------------------------------------
Changes (by ericholscher):

  * needs_tests:  0 => 1
  * stage:  Unreviewed => Accepted

Comment:

 Looks good to me. AFAIK we don't test that ipython works. However, this
 seems like a nice place that mocking would be useful (define an 'ipython'
 on the pythonpath, and confirm that that part of the code is executed
 correctly).

 Since it isn't already tested, I don't know if tests are really necessary
 for checkin, but I know that we try to get parts that aren't tested to
 have tests when they're touched.

-- 
Ticket URL: <http://code.djangoproject.com/ticket/14187#comment:3>
Django <http://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 post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/django-updates?hl=en.

Reply via email to