jakt icon indicating copy to clipboard operation
jakt copied to clipboard

parser: Allow function parameters/arguments to be separated with spaces

Open mattco98 opened this issue 3 years ago • 3 comments

See https://github.com/SerenityOS/jakt/pull/988#issuecomment-1199936147.

mattco98 avatar Jul 30 '22 17:07 mattco98

I don't know about anyone else, but this makes me really uncomfortable. 😬

AtkinsSJ avatar Aug 01 '22 07:08 AtkinsSJ

@AtkinsSJ - the intent here is that you will run Jakt with a formatter that automatically formats your code. Being a bit loose with syntax like allowing you to use a space, allows the formatter to just put it in for you without you having to do it.

sophiajt avatar Aug 01 '22 21:08 sophiajt

@jntrnr I do get your point. Personally though, I prefer the clear separation of a comma/newline over the convenience of not having to type one character. (I'd also have semicolons in the language if it was up to me though. 😅)

AtkinsSJ avatar Aug 02 '22 07:08 AtkinsSJ

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days if no further activity occurs. Thank you for your contributions!

stale[bot] avatar Jan 01 '23 12:01 stale[bot]

This pull request has been closed because it has not had recent activity. Feel free to re-open if you wish to still contribute these changes. Thank you for your contributions!

stale[bot] avatar Jan 09 '23 05:01 stale[bot]