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

add ignored *args to HasTraits constructor #3205

Merged
merged 2 commits into from Apr 26, 2013

Conversation

minrk
Copy link
Member

@minrk minrk commented Apr 21, 2013

makes subclassing nicer / easier, avoiding unclear

TypeError: __new__() takes exactly 1 argument (2 given)

discussed on the list.

@rmcgibbo
Copy link
Contributor

+1

@ellisonbg
Copy link
Member

This looks great, can you add a simple test to make sure that subclasses of HasTraits can use positional args now?

@minrk
Copy link
Member Author

minrk commented Apr 25, 2013

test added

ellisonbg added a commit that referenced this pull request Apr 26, 2013
add ignored *args to HasTraits constructor
@ellisonbg ellisonbg merged commit 46e23e9 into ipython:master Apr 26, 2013
@minrk minrk deleted the traitletsargs branch March 31, 2014 23:36
mattvonrocketstein pushed a commit to mattvonrocketstein/ipython that referenced this pull request Nov 3, 2014
add ignored *args to HasTraits constructor
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

3 participants