History | View | Annotate | Download (4.6 kB)
devel/review: fix addition of reviewer from environment
In commit 3dd990, the possibility of taking a default reviewerfrom the environment was introduced. Fix two deficits here- perl does not inherit the environment- add at all places.
Signed-off-by: Klaus Aehlig <aehlig@google.com>...
devel/review: take default reviewer from environment
When adding a 'Reviewed-by'-line, also add the reviewerpassed through the REVIEWER environment variable. In thatway, adding the reviewer for patch series all reviewedby the same person gets easier....
Use reserved documentation IPs and domains
Use RFC 5737 IP addresses and RFC 2606 domain names in allunittests, docs, qa and docstrings.
Signed-off-by: Manuel Franceschini <livewire@google.com>Reviewed-by: Iustin Pop <iustin@google.com>
devel/review: make the range argument optional
Rather than calling devel/review with from..to target we can just usetarget, and assume target..HEAD as the range. This makes it a loteasier, as now you just have to say something like "devel/reviewdevel-2.1" if you're already on the branch you want to commit, and want...
Implement strict mode for devel/review
This should prevent typos in aliases from going unnoticed.
Signed-off-by: Michael Hanselmann <hansmi@google.com>Reviewed-by: Guido Trotter <ultrotter@google.com>
Add review script
I've been using this script for a while to update commits beforepushing them to the main repository. It copies all commits in arange to another branch using git cherry-pick and starts an editorto modify the Reviewed-by: line(s) for each commit. The script is...