Add -C option to commit patches immediately
Review Request #6122 — Created July 21, 2014 and submitted
Information | |
---|---|
sgallagh | |
RBTools | |
master | |
2a12a09... | |
Reviewers | |
rbtools | |
It is very useful to be able to script a series of patch
applications all at once (such as when reviewing a patch series).
In these cases, it is useful to be able to accept the default
commit message for each patch.
Patched my local RBTools installation with these changes and pulled a patch into my working tree using the -C option. It applied with the default commit as expected.
Description | From | Last Updated |
---|---|---|
Col: 80 E501 line too long (83 > 79 characters) |
![]() |
|
Col: 80 E501 line too long (83 > 79 characters) |
![]() |
|
Col: 80 E501 line too long (83 > 79 characters) |
![]() |
|
I think I'd prefer to have this be called run_editor and have it set to True when -c is passed. |
|
|
The long option should probably be --commit-no-edit |
|
Change Summary:
Fixed long lines detected by Review Bot.
Commit: |
|
||||
---|---|---|---|---|---|
Diff: |
Revision 2 (+24 -7) |

-
Tool: Pyflakes Processed Files: rbtools/commands/patch.py rbtools/clients/__init__.py rbtools/clients/mercurial.py rbtools/clients/git.py Tool: PEP8 Style Checker Processed Files: rbtools/commands/patch.py rbtools/clients/__init__.py rbtools/clients/mercurial.py rbtools/clients/git.py
-
-
rbtools/clients/__init__.py (Diff revision 2) I think I'd prefer to have this be called
run_editor
and have it set to True when -c is passed. -
Commit: |
|
||||
---|---|---|---|---|---|
Diff: |
Revision 3 (+25 -7) |