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 2012年02月06日 08:32 by tshepang, last changed 2022年04月11日 14:57 by admin. This issue is now closed.
| Files | ||||
|---|---|---|---|---|
| File name | Uploaded | Description | Edit | |
| rm-commented-out-code.patch | tshepang, 2012年02月06日 08:32 | review | ||
| Messages (3) | |||
|---|---|---|---|
| msg152727 - (view) | Author: Tshepang Lekhonkhobe (tshepang) * | Date: 2012年02月06日 08:32 | |
It does not appear that the commented-out code adds any value. If anything, it's maybe just a distraction. |
|||
| msg152736 - (view) | Author: Éric Araujo (eric.araujo) * (Python committer) | Date: 2012年02月06日 14:34 | |
Before Tarek started the distutils2 project, the version code lived standalone and its README file served as doctest. Most of its text was moved to PEP 386 and the tests themselves to test_verlib (and now test_version): https://bitbucket.org/tarek/distutilsversion/changeset/140170ef3896#chg-README.txt So you’re right, the lines that your patch removes are indeed unneeded, but I’d just like to check that all tests were moved into test_version before applying it. |
|||
| msg153152 - (view) | Author: Éric Araujo (eric.araujo) * (Python committer) | Date: 2012年02月12日 03:44 | |
Done as part of my general cleanup of test_version. |
|||
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2022年04月11日 14:57:26 | admin | set | github: 58158 |
| 2012年06月25日 12:09:03 | ned.deily | set | messages: - msg163958 |
| 2012年06月25日 12:05:48 | python-dev | set | nosy:
+ python-dev messages: + msg163958 |
| 2012年02月12日 03:44:44 | eric.araujo | set | status: open -> closed resolution: duplicate messages: + msg153152 superseder: Get rid of doctests in packaging.tests.test_version stage: resolved |
| 2012年02月06日 14:34:21 | eric.araujo | set | assignee: tarek -> eric.araujo messages: + msg152736 versions: + 3rd party |
| 2012年02月06日 08:32:26 | tshepang | create | |