search

stil4m / elm-syntax

Elm syntax in Elm
MIT License
92 stars 26 forks source link

Inconsistent naming for arguments #170

Open jfmengels opened 1 year ago

jfmengels commented 1 year ago

For functions (in declarations and let declarations), the arguments are under a field named "arguments". For lambdas, the arguments are under the name "args".

I think we should rename "args" to "arguments" for lambdas.

lenards commented 1 year ago

Jeroen, I realize that this is something someone might already have waiting to PR - but I did a quick rename on this:

I did run elm make --docs=docs.json, but I'm not still seeing "args" within the docs.json, so I might not have that handled correctly.

If the above diff looks like what you expected, I can put it up as a pull request.

lenards commented 1 year ago

(I've been trying to use elm-syntax to help with an i18n extractor; thought I'd try to help out - even if it's really small thing)

🙇