Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support indentation with spaces #111

Merged
merged 10 commits into from
Dec 6, 2024
Merged

Support indentation with spaces #111

merged 10 commits into from
Dec 6, 2024

Conversation

tanzaku
Copy link
Collaborator

@tanzaku tanzaku commented Nov 28, 2024

インデントの方式をタブ文字または空白文字に切り替える indent_tab オプションを追加しました。

resolve #43

@ppputtyo
Copy link
Contributor

ppputtyo commented Dec 4, 2024

uroborosql-fmt/crates/uroborosql-fmt/testfiles/config_test/srctrailing_comment.sqlが存在しないようなのですが、push忘れですかね?
image

@ppputtyo
Copy link
Contributor

ppputtyo commented Dec 4, 2024

indent_tab=false にした場合はすべての区切り文字がスペースになると認識しています。
しかし、以下の2か所でタブが混在すると思われるのですがいかがでしょうか?

Some(format!("{op}\t{all_some_any_keyword}")),

@tanzaku
Copy link
Collaborator Author

tanzaku commented Dec 4, 2024

uroborosql-fmt/crates/uroborosql-fmt/testfiles/config_test/srcにtrailing_comment.sqlが存在しないようなのですが、push忘れですかね?

本来pushするつもりではなかったのですが、誤ってpushしてしまっていたため削除しました。 ad8ba12

indent_tab=false にした場合はすべての区切り文字がスペースになると認識しています。
しかし、以下の2か所でタブが混在すると思われるのですがいかがでしょうか?

修正漏れていました。タブではなくスペースを出力するよう修正しました。 34d85c0

あわせてclippyでエラーになっていた箇所も修正しました 9dd2d84

@abcd-ts abcd-ts merged commit 53f371a into main Dec 6, 2024
8 checks passed
@abcd-ts abcd-ts deleted the support-space-indent branch December 6, 2024 11:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

インデントに使う文字としてスペースを選択できるようにして欲しい
3 participants