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

既存のフォーマッタを導入したい #41

Open
meltingrabbit opened this issue Nov 26, 2021 · 1 comment
Open

既存のフォーマッタを導入したい #41

meltingrabbit opened this issue Nov 26, 2021 · 1 comment
Labels
icebox icebox or pending tools

Comments

@meltingrabbit
Copy link
Collaborator

概要

既存のフォーマッタ,リンタを導入したい

詳細

現在, https://github.com/ut-issl/c2a-core/blob/feature/add_coding_rule_check_ci/.github/workflows/check_coding_rule.yml で,お手製コード規約チェックCIを動かしているが,お手製なので,既存のもの(clang-formatとか?)を動かせるようにしたい.

ただ,clang-formatの使い勝手が以下の点であってなく,問題になっている.

  • ルールを全部決めなくてはならず,一部ルールを無視する設定がない(まあフォーマッタなので妥当ではある)
  • すると,現C2Aのコードに合わせたルールが作れなかった
  • C言語なため,名前空間が切れず,変数名や識別子が長くなりがち
    • 1行の上限長さを設定した場合,小さい値だとめちゃくちゃ改行が発生してむしろ見ずらい
    • 長めに設定すると,改行してもいいようなif文などが全部1行になってしまい,それはそれで見ずらい
  • 細かい辛さ
    • if のワンライナーが改行されて規約違反( {} のない 複数行 if は禁止)になったり
    • defineは改行されたくない

などなど.

もはや上の細かいことは全て諦め,フォーマッタの出力を正義,とする考えもあり.

close条件

フォーマッタ,リンタまわりがなんとかなったら.

@sksat
Copy link
Collaborator

sksat commented May 17, 2023

linter としては実は clang-tidy とか Python script とか clippy とかが既にあるので,問題は formatter

@sksat sksat changed the title 既存のフォーマッタ,リンタを導入したい 既存のリンタを導入したい May 17, 2023
@sksat sksat changed the title 既存のリンタを導入したい 既存のフォーマッタを導入したい May 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
icebox icebox or pending tools
Projects
None yet
Development

No branches or pull requests

2 participants