Don't merge lines if they contain a trailing comma

[ryaminal]

Snowflake now supports trailing commas. See this for more info.

example(from the link above):

select emp_id,
       name,
       dept,
from employees;

Curious on what is desired in sqlfmt to support this? Maybe nothing needs to be done? I'm not certain.

[tconbeer]

when I format your example, I get:

select emp_id, name, dept,
from employees
;

What would you like to see? Do you want to enforce the presence of a trailing comma? Or treat it like black's magic trailing comma (to prevent collapsing onto one line)?

[ryaminal]

mostly just curious if anything needs to be changed from sqlfmt's perspective to support this. didn't want the trailing comma to be removed because it's now valid.
and yeah, i saw the same result and that's why i wasn't certain if anything needs to be done.

[tconbeer]

I don't think we need to change anything, but I'll leave this open as a feature request for a "magic trailing comma" feature