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.

Sign up for GitHub

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

Jump to bottom

Update __repr__ to include Class Name #67

Open
jvanderaa opened this issue Jul 13, 2022 · 1 comment
Open

Update __repr__ to include Class Name #67

jvanderaa opened this issue Jul 13, 2022 · 1 comment
Labels
hacktoberfest Good issue for Hactoberfest status: accepted This issue has been accepted by the maintainers team for implementation status: help wanted This issue is tentatively accepted pending a volunteer committed to its implementation type: minor feature

Comments

@jvanderaa
Copy link
Contributor

Per issue #65...

It would be good to provide a more descriptive __repr__ to not confuse what the object is that is being viewed.
@jvanderaa jvanderaa added status: accepted This issue has been accepted by the maintainers team for implementation status: help wanted This issue is tentatively accepted pending a volunteer committed to its implementation type: minor feature labels Jul 13, 2022
@jvanderaa jvanderaa added the hacktoberfest Good issue for Hactoberfest label Sep 26, 2022
@jvanderaa jvanderaa mentioned this issue Sep 26, 2022
Closed
nautics889 added a commit to nautics889/pynautobot that referenced this issue Apr 1, 2023
@nautics889
fix: Enhancement of __repr__ method (nautobot#67)
8a740b7 
Update `Record` class in pynautobot/core/response.py module.
Add more detailed info the `__repr__()` method returns.
@nautics889
Copy link
Contributor

nautics889 commented Apr 1, 2023 •
edited
Loading

@jvanderaa please have a look at #113

I actually thought about adding id also, but the represetation string would look too long as for me.

nautics889 added a commit to nautics889/pynautobot that referenced this issue Apr 1, 2023
@nautics889
fix: Enhancement of __repr__ method (nautobot#67)
ba83247 
Update `Record` class in pynautobot/core/response.py module.
Add more detailed info the `__repr__()` method returns.
nautics889 added a commit to nautics889/pynautobot that referenced this issue Apr 4, 2023
@nautics889
fix: Enhancement of __repr__ method (nautobot#67)
cc32152 
Update `__repr__()` of `Record` class in pynautobot/core/response.py
module. Add object's memory location to repr string. Removed word
'object'.
nautics889 added a commit to nautics889/pynautobot that referenced this issue Apr 4, 2023
@nautics889
fix: Enhancement of __repr__ method (nautobot#67)
2f8c379 
Update `__repr__()` of `Record` class in pynautobot/core/response.py
module. Get rid of ambiguous meaning of 'name=' in the string which
`__repr__()` returns.
nautics889 added a commit to nautics889/pynautobot that referenced this issue Apr 4, 2023
@nautics889
fix: Enhancement of __repr__ method (nautobot#67)
5d0c642 
Update docstring for `Record` class in pynautobot/core/response.py
module.
jvanderaa added a commit that referenced this issue Apr 10, 2023
@jvanderaa
Merge pull request #113 from nautics889/nautics-67-enhancement-of-repr
4c3bad3 
fix: Enhancement of __repr__ method (#67)
@nautics889 nautics889 mentioned this issue Apr 21, 2023
Closed
jamesharr pushed a commit to jamesharr/pynautobot that referenced this issue Jun 15, 2023
@nautics889
fix: Enhancement of __repr__ method (nautobot#67)
287862f 
Update `Record` class in pynautobot/core/response.py module.
Add more detailed info the `__repr__()` method returns.
jamesharr pushed a commit to jamesharr/pynautobot that referenced this issue Jun 15, 2023
@nautics889
fix: Enhancement of __repr__ method (nautobot#67)
3b5f08c 
Update `__repr__()` of `Record` class in pynautobot/core/response.py
module. Add object's memory location to repr string. Removed word
'object'.
jamesharr pushed a commit to jamesharr/pynautobot that referenced this issue Jun 15, 2023
@nautics889
fix: Enhancement of __repr__ method (nautobot#67)
5160f2e 
Update `__repr__()` of `Record` class in pynautobot/core/response.py
module. Get rid of ambiguous meaning of 'name=' in the string which
`__repr__()` returns.
jamesharr pushed a commit to jamesharr/pynautobot that referenced this issue Jun 15, 2023
@nautics889
fix: Enhancement of __repr__ method (nautobot#67)
fc3f6df 
Update docstring for `Record` class in pynautobot/core/response.py
module.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
hacktoberfest Good issue for Hactoberfest status: accepted This issue has been accepted by the maintainers team for implementation status: help wanted This issue is tentatively accepted pending a volunteer committed to its implementation type: minor feature
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jvanderaa @nautics889