-
-
reviewboard/scmtools/rtc.py (Diff revision 1) Col: 21 E126 continuation line over-indented for hanging indent
-
-
-
-
reviewboard/scmtools/rtc.py (Diff revision 1) Col: 36 E711 comparison to None should be 'if cond is not None
-
-
reviewboard/scmtools/rtc.py (Diff revision 1) Col: 75 E502 the backslash is redundant between brackets
-
reviewboard/scmtools/rtc.py (Diff revision 1) Col: 19 E128 continuation line under-indented for visual indent
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
[reviewboard] Rational Team Concert Support
Review Request #4087 — Created May 1, 2013 and discarded
Patch against release-1.7.7 because master was not work :) Right now integration supports patch compatible with Jazz Patch 1.0 Unfortunately I were not able test it against RTC 4.0.2 server configured with BASIC authentication method. Testing against BASIC authentication method and Websphere will be VERY helpful. While testing patches generated in Eclipse environment was used however it also should works against patches generated by command-line tool. DiffParser base on '#before_state" information to fetch information. Username and password to RTC Repository are required! IMPORTANT!!! User given in repository configuration must be member of all necessery Project Areas and Team Areas, have assigned license and role to be able to read source control content!!! Check credentials and Access Control configuration twice before you raise issue!!! TODO: * Handle "createemptyfile"
* Diff with new files * Diff with changes * Diff with deleted files * Diff with moved files * Against RTC Tomcat with FORM authentication method
| Description | From | Last Updated |
|---|---|---|
|
Col: 21 E126 continuation line over-indented for hanging indent |
 reviewbot
|
|
|
Col: 1 E302 expected 2 blank lines, found 1 |
reviewbot
|
|
|
Col: 80 E501 line too long (81 > 79 characters) |
reviewbot
|
|
|
Col: 5 E303 too many blank lines (2) |
reviewbot
|
|
|
Col: 36 E711 comparison to None should be 'if cond is not None |
reviewbot
|
|
|
Col: 32 E225 missing whitespace around operator |
reviewbot
|
|
|
Col: 75 E502 the backslash is redundant between brackets |
reviewbot
|
|
|
Col: 19 E128 continuation line under-indented for visual indent |
reviewbot
|
|
|
Col: 38 E203 whitespace before ',' |
reviewbot
|
|
|
Col: 1 E302 expected 2 blank lines, found 1 |
reviewbot
|
|
|
Col: 5 E303 too many blank lines (2) |
reviewbot
|
|
|
Col: 80 E501 line too long (108 > 79 characters) |
reviewbot
|
|
|
Col: 1 W293 blank line contains whitespace |
reviewbot
|
|
|
Col: 5 E303 too many blank lines (2) |
reviewbot
|
|
|
Col: 10 E111 indentation is not a multiple of four |
reviewbot
|
|
|
Col: 10 E111 indentation is not a multiple of four |
reviewbot
|
|
|
Col: 10 E111 indentation is not a multiple of four |
reviewbot
|
|
|
Col: 10 E111 indentation is not a multiple of four |
reviewbot
|
|
|
Col: 10 E111 indentation is not a multiple of four |
reviewbot
|
|
|
Col: 10 E111 indentation is not a multiple of four |
reviewbot
|
|
|
Col: 10 E111 indentation is not a multiple of four |
reviewbot
|
|
|
Col: 10 E111 indentation is not a multiple of four |
reviewbot
|
|
|
Col: 10 E111 indentation is not a multiple of four |
reviewbot
|
|
|
Col: 10 E111 indentation is not a multiple of four |
reviewbot
|
|
|
Col: 10 E111 indentation is not a multiple of four |
reviewbot
|
|
|
Col: 10 E111 indentation is not a multiple of four |
reviewbot
|
|
|
Col: 10 E111 indentation is not a multiple of four |
reviewbot
|
|
|
Col: 10 E111 indentation is not a multiple of four |
reviewbot
|
|
|
Col: 10 E111 indentation is not a multiple of four |
reviewbot
|
|
|
Col: 10 E111 indentation is not a multiple of four |
reviewbot
|
|
|
Col: 21 E126 continuation line over-indented for hanging indent |
reviewbot
|
|
|
Col: 36 E711 comparison to None should be 'if cond is not None |
reviewbot
|
|
|
Col: 75 E502 the backslash is redundant between brackets |
reviewbot
|
|
|
Col: 19 E128 continuation line under-indented for visual indent |
reviewbot
|
Change Summary:
Branch/release changes
Description: |
|
||||||||||||||||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Branch: |
|
Description: |
|
|---|
Description: |
|
|---|
Description: |
|
|---|
-
This is a review from Review Bot. Tool: PEP8 Style Checker Processed Files: reviewboard/scmtools/models.py reviewboard/scmtools/admin.py reviewboard/scmtools/core.py reviewboard/scmtools/forms.py reviewboard/scmtools/rtc.py setup.py Ignored Files: reviewboard/templates/admin/repository_fields.js .gitignore -
reviewboard/scmtools/rtc.py (Diff revision 2) Col: 21 E126 continuation line over-indented for hanging indent
-
reviewboard/scmtools/rtc.py (Diff revision 2) Col: 36 E711 comparison to None should be 'if cond is not None
-
reviewboard/scmtools/rtc.py (Diff revision 2) Col: 75 E502 the backslash is redundant between brackets
-
reviewboard/scmtools/rtc.py (Diff revision 2) Col: 19 E128 continuation line under-indented for visual indent
-
This is a review from Review Bot. Tool: PEP8 Style Checker Processed Files: reviewboard/scmtools/models.py reviewboard/scmtools/admin.py reviewboard/scmtools/core.py reviewboard/scmtools/forms.py reviewboard/scmtools/rtc.py setup.py Ignored Files: reviewboard/templates/admin/repository_fields.js .gitignore
-
This is a review from Review Bot. Tool: PEP8 Style Checker Processed Files: reviewboard/scmtools/models.py reviewboard/scmtools/admin.py reviewboard/scmtools/core.py reviewboard/scmtools/forms.py reviewboard/scmtools/rtc.py setup.py Ignored Files: reviewboard/templates/admin/repository_fields.js .gitignore
This is a review from Review Bot. Tool: PEP8 Style Checker Processed Files: reviewboard/scmtools/models.py reviewboard/scmtools/admin.py reviewboard/scmtools/core.py reviewboard/scmtools/forms.py reviewboard/scmtools/rtc.py setup.py Ignored Files: reviewboard/templates/admin/repository_fields.js WARNING: Number of comments exceeded maximum, showing 30 of 31.