Code review comment for lp:~rodsmith/checkbox/intro-docs

Revision history for this message
Brendan Donegan (brendan-donegan) wrote :

On Tue, Sep 16, 2014 at 1:30 PM, Roderick Smith <email address hidden>
wrote:

> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
>
> On 09/16/2014 03:12 AM, Brendan Donegan wrote:
> > Sorry but the form 'CheckBox' still drives me nuts :) It's
> > 'Checkbox' as per the project page: https://launchpad.net/checkbox
> > !
>
> FWIW, I went with CheckBox because that's what's in the Glossary
> (which I did not write) of the document I was editing:
>
> http://plainbox.readthedocs.org/en/latest/glossary.html

Yeah - *somebody* decided to adopt that form in all the documentation they
wrote (looks at Zygmunt :P). Camel-case is just really unnatural and I hate
reading it in natural language. Code is fine because well we need to put up
with all sorts of unnatural structures in code, but camel-case should not
be permeating everywhere.

>
>
> > On Mon, Sep 15, 2014 at 8:06 PM, Zygmunt Krynicki <
> > <email address hidden>> wrote:
> >
> >> On Monday, September 15, 2014, Roderick Smith
> >> <email address hidden> wrote:
> >>
> >>> Added comments on two of Zygmunt's suggested changes.
> >>>
> >>
> >>
> >> Thanks for getting back to that merge request Rod. I'm sorry for
> >> neglecting it myself. We were talking with Daniel about merging
> >> it and doing some post-merge editing but we got lost in our daily
> >> tasks.
> >>
> >> I'll read the updated diff and reply again.
> >>
> >> Thanks ZK
> >>
> >> --
> >> https://code.launchpad.net/~rodsmith/checkbox/intro-docs/+merge/232753
> >>
> >>
> Your team Checkbox Developers is requested to review the proposed merge of
> >> lp:~rodsmith/checkbox/intro-docs into lp:checkbox.
> >>
> >
>
>
> - --
> Rod Smith
> Server and Cloud Certification Engineer
> <email address hidden>
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v1
>
> iQEcBAEBAgAGBQJUGC0OAAoJEFgyRI+V0FjmOrsH/jPkxk0Htt1Yr3Cer5vaf18l
> WgXmGV6m+/Hp/IwvOsFJwHoJlZ7QTs6Ur/5kJWv9PtNIfvRRJ9AeOlL1cdgiYOeP
> XJHKipvZ7AmNR9Mgb9/7x/n7Vmh/ZwBez10/v9b7GGgfwzseId0Ybd0CzXvL4lrJ
> j2e5Qos9diTfrPgJgOQ958HAomkxpVVBukIFcERDlqH42H+3SsnZEdwMTKetUNAn
> D6PyHsl1xZTllABkYS27/axr+jumeYopo0/Xkl3FSjTMVzhsDfvSuaLrfE5AinsQ
> 0oNpHxzJlLyNxCNtAGSkXyGQMF6QAvPhkqAa0k7smMBgWdzSZ7qIHjI4AQGbxRc=
> =FW9X
> -----END PGP SIGNATURE-----
>
> --
> https://code.launchpad.net/~rodsmith/checkbox/intro-docs/+merge/232753
> Your team Checkbox Developers is requested to review the proposed merge of
> lp:~rodsmith/checkbox/intro-docs into lp:checkbox.
>

« Back to merge proposal