Status
People
(Reporter: Pike, Assigned: Pike)
Tracking
Details
Attachments
(1 attachment)
|
21.98 KB,
patch
|
gandalf
:
review+
|
Details | Diff | Splinter Review |
I've got a branch to pep8 lib and tests in python-l20n.
|
(Assignee) | |
Comment 1•6 years ago
|
||
Created attachment 780986 [details] [diff] [review] pep8 lib and tests, didn't touch tools. Also removes obsolete test. pep8-ing tests and lib, and I removed tests/lib.py, cause that tested code that's removed, too. I didn't touch tools, mostly because those aren't candidates for a code-drop to compare-locales, and can happen out-of-band.
Attachment #780986 -
Flags: review?(gandalf)
|
||
Comment 2•6 years ago
|
||
Comment on attachment 780986 [details] [diff] [review] pep8 lib and tests, didn't touch tools. Also removes obsolete test. Review of attachment 780986 [details] [diff] [review]: ----------------------------------------------------------------- yup!
Attachment #780986 -
Flags: review?(gandalf) → review+
|
|
(Assignee) | |
Comment 3•6 years ago
|
||
https://github.com/l20n/python-l20n/commit/1f174d95a2eccbdb857fd43e2e2e3317733c987b, FIXED.
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•