search

kmyk-jikka / Jikka

an automated solver for problems of competitive programming
https://kmyk-jikka.github.io/Jikka/playground
Apache License 2.0
152 stars 11 forks source link

Remove unnecessary paren #143

Closed uta8a closed 2 years ago

uta8a commented 2 years ago

close #130

添字の式は Jikka/CPlusPlus/Format.hsのreturnで返す値部分 と近いと考えてこれを参考に変更しました。

テストを加えたいです。そこで、test/Jikka/CPlusPlus/FormatSpec.hs にテストケースを追加するか、もしくは既存のテストケースに添え字を使うように変更をいれるか迷っています。どちらがよいでしょうか。

kmyk commented 2 years ago

テストを加えたいです。そこで、test/Jikka/CPlusPlus/FormatSpec.hs にテストケースを追加するか、もしくは既存のテストケースに添え字を使うように変更をいれるか迷っています。どちらがよいでしょうか。

どちらでも大丈夫です。どちらかと言えば新しい別のテストケースとして足してもらった方がうれしいかな、くらいの感じです。

uta8a commented 2 years ago

新しいテストケースを追加しました。 (pre-commitを設定しているんですがファイルにtrailing newlineが入ってなかったので直しました...)

kmyk commented 2 years ago

(trailing newline について、pre-commit を設定してるのに直されず CI でも落ちないならもう完全にリポジトリ側の責任です。ごめんね)

kmyk commented 2 years ago

テストの追加ありがとう。ちゃんと直ってるみたいなのでマージします