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 __repr__ to InstallRequirement #2495

Merged
merged 1 commit into from
Mar 6, 2015

Conversation

msabramo
Copy link
Contributor

@msabramo msabramo commented Mar 6, 2015

This makes it easier to inspect with pdb.

Instead of:

(Pdb++) req_to_install
<pip.req.req_install.InstallRequirement object at 0x10459f7d0>

we get the much friendlier:

(Pdb++) req_to_install
<InstallRequirement object: wheel==0.24.0 in /Users/marca/python/virtualenvs/pip/lib/python2.7/site-packages>

This makes it easier to inspect with pdb.

Instead of:

    (Pdb++) req_to_install
    <pip.req.req_install.InstallRequirement object at 0x10459f7d0>

we get the much friendlier:

    (Pdb++) req_to_install
    <InstallRequirement object: wheel==0.24.0 in /Users/marca/python/virtualenvs/pip/lib/python2.7/site-packages>
@msabramo
Copy link
Contributor Author

msabramo commented Mar 6, 2015

Travis CI passed.

dstufft added a commit that referenced this pull request Mar 6, 2015
@dstufft dstufft merged commit 5657013 into pypa:develop Mar 6, 2015
@msabramo msabramo deleted the InstallRequirement_add_repr branch March 6, 2015 16:57
msabramo added a commit to msabramo/pip that referenced this pull request Mar 6, 2015
Corrects my failure to include a test with
pypa#2495
@msabramo
Copy link
Contributor Author

msabramo commented Mar 6, 2015

Oops, forgot to add a test. See #2496

@lock lock bot added the auto-locked Outdated issues that have been locked by automation label Jun 4, 2019
@lock lock bot locked as resolved and limited conversation to collaborators Jun 4, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
auto-locked Outdated issues that have been locked by automation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants