Re: [PATCH v2 00/10] Add 'ls-files --debug-json' to dump the index in json
- From
- Jeff Hostetler <git@jeffhostetler.com>
- Date
- Jun 24, 2019, 18:39 UTC
- Message-ID
- <0367673b-aa5a-49de-87f6-d52beb1af4c4@jeffhostetler.com>
- In-Reply-To
- <nycvar.QRO.7.76.6.1906241954290.44@tvgsbejvaqbjf.bet>
On 6/24/2019 2:00 PM, Johannes Schindelin wrote:
Show 10 quoted lines
> Hi Duy, > > On Mon, 24 Jun 2019, Nguyễn Thái Ngọc Duy wrote: > >> - json field names now use '_' instead of '.' to be friendlier to some >> languages. I stick to underscore_name instead of camelCase because >> the former is closer to what we use > > This is not a good reason. People who are used to read JSON will stumble > over this all the time because it is so uncommon.
Getting rid of "." and "-" in field names is the important part. These confuse some languages and make us drop into object["<field>"] syntax in my experience.
As for "_" or camelCase (or PascalCase), I'm not sure it matters one way or the other. Personally, I'd vote for underscores.
Jeff