-
Notifications
You must be signed in to change notification settings - Fork 95
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
builtin: add dir builtin #20
base: main
Are you sure you want to change the base?
Conversation
Fixes #12.
this is a WIP. @ncw, for |
The way I've done that in the past, eg for implementing locals() is to implement them as InternalMethods. Eg here is where the magic is... Lines 1578 to 1600 in eaa7d28
This is where the other magic is!
That is the only way I could find to avoid using global variables etc. It might be too limiting eventually. |
@sbinet after the |
@sbinet lmk if this is worth adding/improving so we can work towards a cleared PR section. Also, any suggestions on the next version/tag? I think we're worthy of a 0.1 or 1.0 -- open to discussion. |
in order to get the nice discoverability look and feel of CPython into gpython, having a working but as we survived w/o it for almost 4 years, I don't think cleaning up this PR should be a blocker for the next tagged version of gpython. |
Fixes #12.