Support leading commas

[derycck]

This is not a feature request. I would like to know if you accept a pull request to implement "Support leading commas" feature.

I have successfully implemented the "comma_style" flag, where it is possible to choose between "leading" and "trailing", achieving the expected effect in formatting.

To open a pull request, I need to create test cases that support this new feature.

Therefore, to avoid unnecessary work, I would like to know in advance if:

1. you would accept a pull request with the implementation of this feature,
2. or if you are determined to keep the project with a fixed style.

I am very grateful for your project. Thanks.

[tconbeer]

Thanks for starting this discussion.

I'm glad you made this work for you, but no, I'm not open to supporting leading commas in this project. I think trailing commas are better and I don't think teams should have to decide how they configure sqlfmt. But I'm happy to leave this discussion open; if you want to maintain and promote your fork here, that's fine by me.

[derycck]

The fork bellow of sqlfmt implements the option to format SQL in leading comma style. This feature is probably applicable by ongoing projects that already implement leading comma style, improving code readability and maintaining a consistent style in SQL queries.

https://github.com/derycck/sqlfmt

It is a version with no maintenance commitment, for the sake of sharing something that has been useful to me.

[svdimchenko]

@derycck thx a lot for sharing your implementation 💯 🌟
We have 1500+ dbt models with leading comma code style, I'm applying your fork to be run as pre-commit hook. It works for some models, will share insights or contribute if needed