Issue32173
Created on 2017-11-30 02:57 by ganziqim, last changed 2022-04-11 14:58 by admin. This issue is now closed.
| Pull Requests | |||
|---|---|---|---|
| URL | Status | Linked | Edit |
| PR 4603 | closed | ganziqim, 2017-11-30 02:57 | |
| PR 4641 | merged | ganziqim, 2017-11-30 03:12 | |
| PR 10130 | closed | thatiparthy, 2018-10-26 15:22 | |
| Messages (5) | |||
|---|---|---|---|
| msg307406 - (view) | Author: Raymond Hettinger (rhettinger) * | Date: 2017-12-01 19:46 | |
These seems to be reasonable suggestions. Lisa, would you care to review the patch? |
|||
| msg307409 - (view) | Author: Serhiy Storchaka (serhiy.storchaka) * | Date: 2017-12-01 20:04 | |
Could you please explain your suggestion ganziqim? The patch looks like a collection of unrelated changes mostly changing the formatting of the code. Usually we don't do such changes. If there is a semantical change, it is hidden in a source churn. |
|||
| msg307419 - (view) | Author: Lisa Roach (lisroach) * | Date: 2017-12-02 02:45 | |
I think these changes look good, ganziqim has real code improvements (in the form of adding lazycache to __all__ and using clear() in clearchache), as well as improving the code formatting for readability. |
|||
| msg328571 - (view) | Author: Serhiy Storchaka (serhiy.storchaka) * | Date: 2018-10-26 16:57 | |
Since all proposed PRs just change formatting, I suggest to close this issue. |
|||
| msg364724 - (view) | Author: A.M. Kuchling (akuchling) * | Date: 2020-03-21 00:28 | |
Patch has been applied. |
|||
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2022-04-11 14:58:55 | admin | set | github: 76354 |
| 2020-03-21 00:28:33 | akuchling | set | status: open -> closed nosy:
+ akuchling resolution: fixed |
| 2018-10-26 16:57:40 | serhiy.storchaka | set | messages: + msg328571 |
| 2018-10-26 15:22:04 | thatiparthy | set | pull_requests: + pull_request9462 |
| 2017-12-02 02:45:54 | lisroach | set | messages: + msg307419 |
| 2017-12-01 20:04:51 | serhiy.storchaka | set | nosy:
+ serhiy.storchaka messages: + msg307409 |
| 2017-12-01 19:46:54 | rhettinger | set | assignee: lisroach messages:
+ msg307406 |
| 2017-11-30 03:12:26 | ganziqim | set | keywords:
+ patch stage: patch review pull_requests: + pull_request4555 |
| 2017-11-30 02:57:02 | ganziqim | create | |