Closed harlanhaskins closed 6 years ago
This avoids mixing cases in command-line arguments. We don't want kebab-case mixed with camelCase.
kebab-case
camelCase
Taking from silgen, I think girgen is one word, with no separator.
silgen
girgen
(This also adds a comment above two uncommented dump layers)
Consistency.
Should we split it into gir-gen?
gir-gen
¯_(ツ)_/¯
⛵️
CodaFi
commented
6 years ago CodaFi
commented
6 years ago
What's in this pull request?
This avoids mixing cases in command-line arguments. We don't want
kebab-casemixed withcamelCase.Taking from
silgen, I thinkgirgenis one word, with no separator.(This also adds a comment above two uncommented dump layers)
Why merge this pull request?
Consistency.
What's worth discussing about this pull request?
Should we split it into
gir-gen?What downsides are there to merging this pull request?
¯_(ツ)_/¯