Review Request #9512 — Created Jan. 20, 2018 and updated
| Information | |
|---|---|
| jque | |
| Review Board | |
| release-2.0.x | |
| 3c76a5c... | |
| Reviewers | |
| reviewboard, students | |
Filenames in git diff files should be relative path and not
absolute path. This EasyFix raises an error when an
absolute path is detected.
Added unit test to test parsing Git diff with absolute file names
All tests passed in reviewboard.scmtools.tests.test_git:GitTests
| Description | From | Last Updated |
|---|---|---|
|
Does this have an associated ticket on Splat? If so, can you link it? If not (actually, even if so), … |
 david
|
|
|
E501 line too long (81 > 79 characters) |
 reviewbot
|
|
|
E501 line too long (83 > 79 characters) |
reviewbot
|
|
|
E501 line too long (83 > 79 characters) |
reviewbot
|
|
|
E127 continuation line over-indented for visual indent |
reviewbot
|
|
|
E501 line too long (83 > 79 characters) |
reviewbot
|
|
|
E127 continuation line over-indented for visual indent |
reviewbot
|
| reviewboard/scmtools/git.py (Diff revision 1) |
|---|
| reviewboard/scmtools/git.py (Diff revision 1) |
|---|
E127 continuation line over-indented for visual indent
| reviewboard/scmtools/git.py (Diff revision 1) |
|---|
E127 continuation line over-indented for visual indent
Commit: |
|
||||
|---|---|---|---|---|---|
Diff: |
Revision 2 (+32 -4) |
