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

IPython Console: Simple fix after PR #3641 #3660

Merged
merged 1 commit into from Nov 7, 2016

Conversation

ccordoba12
Copy link
Member

@mariacamilaremolinagutierrez, you should have detected this error if you had tested live (i.e. by starting a new instance of Spyder) the commit that introduced this change.

That's why I told you and @dalthviz to never push any change that hasn't been tested live. Let's hope this doesn't happen again :-)


This was (in part) also my fault because I wasn't sure if copy() works for lists or not, but I trusted you did the check yourself ;-)

@ccordoba12 ccordoba12 added this to the v3.0.2 milestone Nov 7, 2016
@ccordoba12 ccordoba12 merged commit f31883d into spyder-ide:3.x Nov 7, 2016
@ccordoba12 ccordoba12 deleted the fix-3641 branch November 7, 2016 19:31
ccordoba12 added a commit that referenced this pull request Nov 7, 2016
Simple fix after PR #3641
@mariacamilarg
Copy link
Contributor

Hi @ccordoba12 I truly apologize for this. I did tested this live, however I only did it in python 3 and copy() for lists does work there. Next time I'll remember to test it in both environments (python 2 and 3) so that this is not repeated. Again, my apologies!

@ccordoba12
Copy link
Member Author

No worries, just please be more careful next time :-)

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.

None yet

2 participants