R= is not necessary anymore. Add “why” to the FAQ. |
|||||||
Issue descriptionNew users will be confused why R= line isn't added to Gerrit reviews if -r option is passed. Perhaps a comment should be added, e.g. # Enter a description of the change. # This will be displayed on the codereview site. # The first line will also be used as the subject of the review. #--------------------This line is 72 characters long-------------------- Foo bar CL description BUG= #----------------------------------------------------------------------- R=foo@chromium.org line is not necessary anymore and Reviewed-By: food@chromium.org will instead be added after the CL is committed
,
Jul 7 2016
,
Aug 3 2016
,
Sep 27 2016
My guess is that no one will read the footer. We should still add it to the docs.
,
Jan 13 2017
+ Add documentation saying R= not necessary. + Modify git cl to remove it from the commit description. + Get other FOO= out of the description. Tranform it to Foo: in the footer. + Get rid of TBR= text-based check in presubmit. + Remove mentions of tags.
,
Jan 3 2018
Removing Milestone-Afterglow, as it has ceased to have meaning. More refined milestones may be added back in the near future.
,
Jan 5 2018
|
|||||||
►
Sign in to add a comment |
|||||||
Comment 1 by tandrii@chromium.org
, Jul 7 2016Status: Available (was: Untriaged)