Code review comment for lp:~sinzui/juju-core/goyaml-50-deps

Revision history for this message
Curtis Hovey (sinzui) wrote :

Reviewers: mp+195428_code.launchpad.net,

Message:
Please take a look.

Description:
Restore YAML block and flow interpretation.

Update the goyaml dep to r50 so that --- and ... can have text
following them.

I verified dependencies.tsv with
     godeps -u dependencies.tsv

https://code.launchpad.net/~sinzui/juju-core/goyaml-50-deps/+merge/195428

(do not edit description out of merge proposal)

Please review this at https://codereview.appspot.com/26960043/

Affected files (+3, -1 lines):
   A [revision details]
   M dependencies.tsv

Index: [revision details]
=== added file '[revision details]'
--- [revision details] 2012-01-01 00:00:00 +0000
+++ [revision details] 2012-01-01 00:00:00 +0000
@@ -0,0 +1,2 @@
+Old revision: tarmac-20131115152216-393eaqgqwsnw9q7o
+New revision: <email address hidden>

Index: dependencies.tsv
=== modified file 'dependencies.tsv'
--- dependencies.tsv 2013-11-14 15:08:13 +0000
+++ dependencies.tsv 2013-11-15 17:19:43 +0000
@@ -7,7 +7,7 @@
  launchpad.net/golxc bzr
<email address hidden> 5
  launchpad.net/gomaasapi bzr
<email address hidden> 47
  launchpad.net/goose bzr tarmac-20131101034733-zbirap24k6k8wjlg 110
-launchpad.net/goyaml bzr
<email address hidden> 49
+launchpad.net/goyaml bzr
<email address hidden> 50
  launchpad.net/gwacl bzr tarmac-20131031081035-b33m6fyrrdiuf408 229
  launchpad.net/loggo bzr
<email address hidden> 39
  launchpad.net/lpad bzr
<email address hidden> 64

« Back to merge proposal