Skip to content
This repository was archived by the owner on Feb 24, 2020. It is now read-only.

Conversation

@achanda
Copy link
Contributor

@achanda achanda commented Aug 7, 2016

We were using the chain name as comment. This can lead to confusion
and does not add much value since the chain name is partly random.

We were using the chain name as comment. This can lead to confusion
and does not add much value since the chain name is partly random.
@ghost
Copy link

ghost commented Aug 7, 2016

Can one of the admins verify this patch?

@lucab
Copy link
Member

lucab commented Aug 7, 2016

ok to test

@lucab lucab added this to the v1.13.0 milestone Aug 7, 2016
@lucab
Copy link
Member

lucab commented Aug 7, 2016

/cc @steveej @jellonek @squeed

@jonboulle
Copy link
Contributor

@coreos/rkt-kvm-maintainers

@jellonek
Copy link
Contributor

jellonek commented Aug 8, 2016

LGTM

@lucab lucab merged commit 4e36fae into rkt:master Aug 9, 2016
@achanda achanda deleted the comment branch August 9, 2016 14:16
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants