Opened 8 years ago
Closed 8 years ago
#2690 closed enhancement (fixed)
PEP8 like code styling for the i.tasscap.py script
| Reported by: | Nikos Alexandris | Owned by: | |
|---|---|---|---|
| Priority: | trivial | Milestone: | 7.0.1 |
| Component: | Python | Version: | unspecified |
| Keywords: | i.tassccap, python, script | Cc: | |
| CPU: | Unspecified | Platform: | Unspecified |
Description
See attached diff.
Attachments (3)
Change History (6)
by , 8 years ago
| Attachment: | i.tasscap.py.diff added |
|---|
follow-up: 2 comment:1 by , 8 years ago
by , 8 years ago
| Attachment: | i.tasscap.py.diff_ignore-all-space added |
|---|
by , 8 years ago
| Attachment: | i.tasscap.py.diff_ignore-space-change added |
|---|
comment:2 by , 8 years ago
Replying to wenzeslaus:
Seems OK. Did you check what you get from
svn diff --ignore-space-changeorsvn diff --ignore-all-space?
Attached diffs derived from svn diff -x '--ignore-all-space' and svn diff -x '--ignore-space-change'.
I would ask you for simple test which would prove that the basic functionality is OK but it would be probably absurd to require it when we lack tests for far more invasive changes.
At minimum, I have tested the script before submitting the (first) diff and it work-ed/-s for me.
Note:
See TracTickets
for help on using tickets.
Seems OK. Did you check what you get from
svn diff --ignore-space-changeorsvn diff --ignore-all-space?I was not sure about the header but now I see that it fits with wiki:Submitting/Python. I'm glad that to learn that. I wonder how it looks like when it gets through Sphinx (not that we would doing that).
I would ask you for simple test which would prove that the basic functionality is OK but it would be probably absurd to require it when we lack tests for far more invasive changes.