➜
This issue tracker has been migrated to GitHub,
and is currently read-only.
For more information,
see the GitHub FAQs in the Python's Developer Guide.
Created on 2010-04-10 13:13 by exarkun, last changed 2022-04-11 14:56 by admin. This issue is now closed.
| Files | ||||
|---|---|---|---|---|
| File name | Uploaded | Description | Edit | |
| test_inspect.patch | exarkun, 2010-04-10 13:14 | Refactor the two tests to use a common helper | ||
| Messages (5) | |||
|---|---|---|---|
| msg102776 - (view) | Author: Jean-Paul Calderone (exarkun) *  |
Date: 2010-04-10 13:13 | |
There are two tests for the way inspect.classify_class_attrs handles various sorts of attributes. The tests are identical, except one uses a classic class and one a new-style class. The tests sources have actually begun to diverge, but so far only in whitespace. This code will be easier to maintain (not to mention shorter) if there is just one copy of the actual test parts. |
|||
| msg102784 - (view) | Author: Benjamin Peterson (benjamin.peterson) * |
Date: 2010-04-10 16:07 | |
I think the patch is ok except for that docstrings should start on the same line as the triple quote. |
|||
| msg102785 - (view) | Author: Jean-Paul Calderone (exarkun) * |
Date: 2010-04-10 16:12 | |
Okay. I shouldn't commit until after trunk is unfrozen though, right? |
|||
| msg102786 - (view) | Author: Benjamin Peterson (benjamin.peterson) * |
Date: 2010-04-10 16:18 | |
2010/4/10 Jean-Paul Calderone <report@bugs.python.org>: > > Jean-Paul Calderone <exarkun@divmod.com> added the comment: > > Okay. > > I shouldn't commit until after trunk is unfrozen though, right? Correct. |
|||
| msg102791 - (view) | Author: Jean-Paul Calderone (exarkun) * |
Date: 2010-04-10 20:03 | |
Committed in r79935 |
|||
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2022-04-11 14:56:59 | admin | set | github: 52610 |
| 2010-04-10 20:03:50 | exarkun | set | status: open -> closed resolution: fixed messages: + msg102791 stage: resolved |
| 2010-04-10 16:18:30 | benjamin.peterson | set | messages: + msg102786 |
| 2010-04-10 16:12:54 | exarkun | set | messages: + msg102785 |
| 2010-04-10 16:07:40 | benjamin.peterson | set | nosy:
+ benjamin.peterson messages: + msg102784 |
| 2010-04-10 13:14:19 | exarkun | set | files: + test_inspect.patch |
| 2010-04-10 13:13:12 | exarkun | create | |