Change Summary:
fix description
Description: |
|
---|
Review Request #5184 — Created Jan. 3, 2014 and submitted
Information | |
---|---|
delyn | |
RBTools | |
master | |
3183 | |
Reviewers | |
rbtools | |
jan.koprowski |
replace revision /streamname/0 by their clearcase predecessor ie parentstream/N (N is origin revision of file in parent stream)
linked with https://reviews.reviewboard.org/r/5168
create clearcase review request using --ucm-activity in a fresh stream ie containing /streamname/0 versions
Description | From | Last Updated |
---|---|---|
'file' is a reserved word in python. This will work, but in order to be more correct, can we call … |
|
|
Sentences in comments should start with a capital letter and end in a period. |
|
|
Please pass format arguments as additional arguments to logging.* |
|
|
This looks like it could be one line and still be less than 79 columns. |
|
|
Please pass format arguments as additional arguments to logging.* |
|
|
Put a blank line between the first line of a docstring (the summary) and the rest. The final """ should … |
|
|
Please wrap this to 79 columns. Also, sentences in comments should start with a capital letter and end with a … |
|
fix description
Description: |
|
---|
rbtools/clients/clearcase.py (Diff revision 1) |
---|
'file' is a reserved word in python. This will work, but in order to be more correct, can we call this something like 'filename' or 'path'?
rbtools/clients/clearcase.py (Diff revision 1) |
---|
Sentences in comments should start with a capital letter and end in a period.
rbtools/clients/clearcase.py (Diff revision 1) |
---|
Please pass format arguments as additional arguments to
logging.*
rbtools/clients/clearcase.py (Diff revision 1) |
---|
This looks like it could be one line and still be less than 79 columns.
rbtools/clients/clearcase.py (Diff revision 1) |
---|
Please pass format arguments as additional arguments to
logging.*
rbtools/clients/clearcase.py (Diff revision 1) |
---|
Put a blank line between the first line of a docstring (the summary) and the rest.
The final """ should also go on its own line.
rbtools/clients/clearcase.py (Diff revision 1) |
---|
Please wrap this to 79 columns. Also, sentences in comments should start with a capital letter and end with a period.