Skip to content
This repository was archived by the owner on Feb 2, 2019. It is now read-only.

Feat: repr() uses class name and Travis-CI fixes#46

Open
alastairmccormack wants to merge 4 commits intobcj:masterfrom
alastairmccormack:feature/repr_subclass_name
Open

Feat: repr() uses class name and Travis-CI fixes#46
alastairmccormack wants to merge 4 commits intobcj:masterfrom
alastairmccormack:feature/repr_subclass_name

Conversation

@alastairmccormack
Copy link
Copy Markdown

I regularly use AttrDict to quickly convert JSON objects into what looks like native objects.

When getting the repr, it's nicer to see the subclassed name rather than AttrDict.

E.g.

>>> class MySubClass(AttrDict):
>>>    pass

>>> msc = AttrDict({})
>>> print(msc)
'MySubClass({})'

@alastairmccormack alastairmccormack changed the title Feat: repr() uses class name. Useful when subclassing Attrdict. Feat: repr() uses class name Feb 27, 2018
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 100.0% when pulling a7421d9 on use-sparingly:feature/repr_subclass_name into 9f67299 on bcj:master.

@coveralls
Copy link
Copy Markdown

coveralls commented Feb 27, 2018

Coverage Status

Coverage remained the same at 100.0% when pulling f6b6fde on use-sparingly:feature/repr_subclass_name into 9f67299 on bcj:master.

alastair.mccormack added 2 commits February 27, 2018 21:43
Support for dependent libraries of nose and Flake 8 in Py 2.6 and Py 3.2
is being removed.

Example:

Requests: psf/requests#3479
@alastairmccormack alastairmccormack changed the title Feat: repr() uses class name Feat: repr() uses class name and Travis-CI fixes Feb 27, 2018
@alastairmccormack
Copy link
Copy Markdown
Author

To make the Travis CI build pass I've had to drop Py 3.2 and 2.6 tests. It seems that many libraries are dropping support which is causing pip install to fail on these versions.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants