lassik / emacs-format-all-the-code

Auto-format source code in many languages with one command
https://melpa.org/#/format-all
MIT License
611 stars 106 forks source link

change sql formatter to sql-formatter #223

Open Tokubara opened 1 year ago

Tokubara commented 1 year ago

The default sql formatter sqlformat doesn't work. For example:

CREATE TABLE student (
sid INT PRIMARY KEY, name VARCHAR(16), login VARCHAR(32) UNIQUE, age SMALLINT, gpa FLOAT );

format this piece of sql with

sqlformat tmp.sql  --reindent --indent_width 4 --keywords upper --use_space_around_operators

gets:

CREATE TABLE student (sid INT PRIMARY KEY,
                                      name VARCHAR(16),
                                           login VARCHAR(32) UNIQUE,
                                                             age SMALLINT, gpa FLOAT);

Without any options, sqlformat tmp.sql just prints the input.

The comment in the first answer about sqlformat points out the same thing.

While sql-formatter works great without any options. So I change sqlformat support to sql-formatter.

lassik commented 1 year ago

We should keep supporting sqlformat. But I agree it's confusing that it does not do anything without any options.

Please edit the PR so that it: