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

Include keys as attributes returned by __dir__() #46

Merged
merged 2 commits into from
Mar 11, 2015
Merged

Include keys as attributes returned by __dir__() #46

merged 2 commits into from
Mar 11, 2015

Conversation

Who8MyLunch
Copy link
Contributor

Updates to dir method to address issues with autocomplete on Dict keys. See #45 for details.

@mewwts
Copy link
Owner

mewwts commented Mar 10, 2015

Hi @Who8MyLunch, thanks for submitting this! 👍 Could you take a moment and update the failing test?

@Who8MyLunch
Copy link
Contributor Author

The failing test has been updated to reflect the new dir behavior.

Pierre Villeneuve
pierre.villeneuve@gmail.com

On Tue, Mar 10, 2015 at 2:39 PM, Mats Julian Olsen <notifications@github.com

wrote:

Hi @Who8MyLunch https://github.com/Who8MyLunch, thanks for submitting
this! [image: 👍] Could you take a moment and update the failing test?


Reply to this email directly or view it on GitHub
#46 (comment).

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 27e1a5e on Who8MyLunch:master into ae78895 on mewwts:master.

@sabhiram
Copy link
Contributor

Cool beans :)

@mewwts mewwts merged commit 27e1a5e into mewwts:master Mar 11, 2015
@mewwts
Copy link
Owner

mewwts commented Mar 11, 2015

I think this looks good! Thanks @Who8MyLunch for contributing, I hope you continue to do it! Feel free to contribute with ideas to #26.

This fixes #45.

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

4 participants