Stowage/go
2
0
Fork 0
mirror of https://github.com/golang/go.git synced 2025-11-09 05:01:01 +00:00
Code Issues Projects Releases Packages Wiki Activity

make hg mail more forgiving:

Browse source 
create CL first, even if gofmt is needed
or no reviewers listed.

R=r
http://go/go-review/1025025
This commit is contained in:
Russ Cox 2009-11-08 22:13:10 -08:00
parent 6fff7b2399
commit b1a52ce4a3
1 changed files with 7 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

9
lib/codereview/codereview.py
View file

@ -896,9 +896,9 @@ def mail(ui, repo, *pats, **opts):
cl, err = CommandLineCL(ui, repo, pats, opts) cl, err = CommandLineCL(ui, repo, pats, opts)
if err != "": if err != "":
return err return err
cl.Upload(ui, repo, gofmt_just_warn=True)
if not cl.reviewer: if not cl.reviewer:
return "no reviewers listed in CL" return "no reviewers listed in CL"
cl.Upload(ui, repo)
pmsg = "Hello " + JoinComma(cl.reviewer) pmsg = "Hello " + JoinComma(cl.reviewer)
if cl.cc: if cl.cc:
pmsg += " (cc: %s)" % (', '.join(cl.cc),) pmsg += " (cc: %s)" % (', '.join(cl.cc),)
@ -1002,7 +1002,12 @@ def submit(ui, repo, *pats, **opts):
# upload, to sync current patch and also get change number if CL is new. # upload, to sync current patch and also get change number if CL is new.
if not cl.original_author: if not cl.original_author:
cl.Upload(ui, repo) cl.Upload(ui, repo, gofmt_just_warn=True)
# check gofmt for real; allowed upload to warn in order to save CL.
cl.Flush(ui, repo)
CheckGofmt(ui, repo, cl.files)
about += "%s%s\n" % (server_url_base, cl.name) about += "%s%s\n" % (server_url_base, cl.name)
if cl.original_author: if cl.original_author: