mesos-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Timothy Chen" <tnac...@apache.org>
Subject Re: Review Request 32001: Required a period in trailing comments in the style guide.
Date Mon, 23 Mar 2015 23:51:53 GMT


> On March 12, 2015, 9:20 p.m., Ben Mahler wrote:
> > docs/mesos-c++-style-guide.md, line 30
> > <https://reviews.apache.org/r/32001/diff/1/?file=892420#file892420line30>
> >
> >     Hm, isn't this captured by ending each sentence with a period above?
> 
> Alexander Rukletsov wrote:
>     Yes and no. Trailing comments are not always sentences (at least in my non-native
speaker's language model), it can be for example a JIRA ticket. As an alternative we can re-phrase
the statement above to something like "Each comment and sentences within it must end with
a period." What do you think?
> 
> Michael Park wrote:
>     Haha I didn't even catch that: "End each __sentence__ with a period." Do we just
mean "End each comment with a period."?
> 
> Alexander Rukletsov wrote:
>     Well, techically a comment may consist of multiple sentences, which means you can
write something like this:
>     ```
>         // To compensate for the case where a non-strict registrar is
>         // being used, we explicitly deny removed slaves from
>         // re-registering
>         // This is because a non-strict registrar cannot
>         // enforce this
>         // We've already told frameworks the tasks were
>         // lost so it's important to deny the slave from re-registering.
>     ```
>     But maybe we lose too much time on this minor thing.
> 
> Niklas Nielsen wrote:
>     I would support mpark's idea here: How about having an extra bullet with "End each
sentence with a period."?

To be technically correct, we need a end each sentence with some puncatuation, as you'll also
commonly see this in our code:

master_tests.cpp
// Step 7: Ensure the slave cannot re-register!


- Timothy


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/32001/#review76286
-----------------------------------------------------------


On March 12, 2015, 8:59 p.m., Alexander Rukletsov wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/32001/
> -----------------------------------------------------------
> 
> (Updated March 12, 2015, 8:59 p.m.)
> 
> 
> Review request for mesos and Ben Mahler.
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> See summary.
> 
> 
> Diffs
> -----
> 
>   docs/mesos-c++-style-guide.md 439fe12ad137c1bed3a21d5a097c60a48f10a4f9 
> 
> Diff: https://reviews.apache.org/r/32001/diff/
> 
> 
> Testing
> -------
> 
> None (not a functional change).
> 
> 
> Thanks,
> 
> Alexander Rukletsov
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message