Replace tools/myvn with new Git workflow helper. (issue 285550043 by kpreid@google.com)
7 views
Skip to first unread message
re...@codereview-hr.appspotmail.com
unread,
Feb 19, 2016, 6:21:42 PM2/19/16
Reply to author
Sign in to reply to author
Forward
Sign in to forward
Delete
You do not have permission to delete messages in this group
Copy link
Report message
Sign in to report message
Show original message
Either email addresses are anonymous for this group or you need the view member email addresses permission to view the original message
to eri...@gmail.com, kpr...@google.com, google-ca...@googlegroups.com, re...@codereview-hr.appspotmail.com
Reviewers: MarkM,
Description:
'tools/myvn change' is now 'tools/workflow prepare'.
'tools/myvn snapshot' is now 'tools/workflow snapshot'.
'tools/myvn eclipse' is now 'tools/eclipse-setup'.
Other stuff does not map as well.
https://codereview.appspot.com/285550043/diff/1/tools/eclipse-setup#newcode23 tools/eclipse-setup:23: echo " <classpathentry
excluding=\"**/*~|**/.svn/**|nu/validator/htmlparser/dom/|
nu/validator/htmlparser/extra/|nu/validator/htmlparser/io/|
nu/validator/htmlparser/sax/|nu/validator/htmlparser/xom/\"
kind=\"src\" path=\"third_party/java/htmlparser/src\"/>"
Please reformat to a narrower width. Ideally 80 columns, but please no
more than 100.
You do not have permission to delete messages in this group
Copy link
Report message
Sign in to report message
Show original message
Either email addresses are anonymous for this group or you need the view member email addresses permission to view the original message
to eri...@gmail.com, kpr...@google.com, google-ca...@googlegroups.com, re...@codereview-hr.appspotmail.com
'tools/myvn change' is now 'tools/workflow prepare'.
'tools/myvn snapshot' is now 'tools/workflow snapshot'.
'tools/myvn eclipse' is now 'tools/eclipse-setup'.
Other stuff does not map as well.
You do not have permission to delete messages in this group
Copy link
Report message
Sign in to report message
Show original message
Either email addresses are anonymous for this group or you need the view member email addresses permission to view the original message
to eri...@gmail.com, kpr...@google.com, google-ca...@googlegroups.com, re...@codereview-hr.appspotmail.com
'tools/myvn change' is now 'tools/workflow prepare'.
'tools/myvn snapshot' is now 'tools/workflow snapshot'.
'tools/myvn eclipse' is now 'tools/eclipse-setup'.
Other stuff does not map as well.
On 2016/02/20 01:56:30, MarkM wrote:
> Please reformat to a narrower width. Ideally 80 columns, but please no
more than
> 100.
This is moved, not new code — it's only in a new file because I split
tools/myvn's job into tools/workflow and this file. I'd rather not
rework and test it as part of this change, since introducing line breaks
is nontrivial due to quoting.
https://codereview.appspot.com/285550043/diff/40001/tools/workflow#newcode44 tools/workflow:44: # $ git merge --ff-only <my-branch-name>
When I got here, I could not remember what branch name I used. I
remembered "git branch" and look for the star. But this happened after
"git checkout master". You should explain how to get your branch name.
Since "tools/workflow push" doesn't work yet, for those who can "git
push" you should mention that in the workflow text above. Probably the
same for a pull request, but I don't know.