On Fri, Jun 10, 2016 at 3:02 PM, Thiago Macieira <thiago.macieira at intel.com>
wrote:

> On sexta-feira, 10 de junho de 2016 14:55:44 PDT Gregg Reynolds wrote:
> >
>
...

>  >
> > > Hi Gregg
> > >
> > > Just make them public (non-draft) state, so everyone can look at it.
> >
> > Hi Thiago,
> >
> > How do I do that?  Click the "Publish" button?
>
> Yes. As a rule of thumb, I recommend not to use the draft feature. Hiding
> code
> is usually not a good idea.
>
> If you want to let people know that it shouldn't be merged, vote -1
> yourself
> or start the commit message with "WIP" (work in progress) or "DO NOT
> SUBMIT".
>
> > The reason I made it a draft is because it seems like a fairly major
> change
> > that should be discussed.  Maybe I'm not understanding the purpose of
> > Drafts.
>
> Probably. It's kind of a misfeature, in my opinion.
>
> You can use it to push things that you want to discuss with a very
> restricted
> set of people. If you post about it to the mailing list, though, it's not
> very
> restricted. Let everyone see it.
>

Thanks, will do.

G
-------------- next part --------------
An HTML attachment was scrubbed...
URL: 
<http://lists.iotivity.org/pipermail/iotivity-dev/attachments/20160610/c8f60b8c/attachment.html>

Reply via email to