Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Always install ipython on prod environments #14794

Merged
merged 1 commit into from
Feb 8, 2017
Merged

Always install ipython on prod environments #14794

merged 1 commit into from
Feb 8, 2017

Conversation

esoergel
Copy link
Contributor

@esoergel esoergel commented Feb 7, 2017

If I ever want to run a django shell, I want ipython available. I don't
think there's a real downside to this, but interested to hear if anyone
disagrees.

If I ever want to run a django shell, I want ipython available.  I don't
think there's a real downside to this, but interested to hear if anyone
disagrees.
@esoergel esoergel changed the title Always install ipython Always install ipython on prod environments Feb 7, 2017
@@ -15,7 +15,7 @@ fixture
sniffer
psutil # for memory profiling
django-extensions
ipython==4.2.1
ipython==5.2.2
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this doesn't work with ipdb do you think we can move this version back down?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are you sure? I just tried it out and it works for me. Could be a mac vs linux thing. If that's the case, then yeah, we'll probably have to go with 4.x

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FYI, I did find some issues with ipdb and ipython5, but it looks like they've been resolved: gotcha/ipdb#108

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes! nice that was the error i was getting. auto complete also wasn't working unless i upgraded ipdb

@benrudolph benrudolph merged commit b93ad4f into master Feb 8, 2017
@benrudolph benrudolph deleted the es/ipython branch February 8, 2017 22:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants