aiscript icon indicating copy to clipboard operation
aiscript copied to clipboard

各構文で改行できる箇所の見直し

Open marihachi opened this issue 2 years ago • 5 comments

新パーサーでも既存のテストは通るようになりました。 ただ、ここでは改行したい!という部分がまだまだ残ってる可能性はあります。 その辺を一通り見直していきたい。

ローカルでplaygroundを立ち上げれば(npm run dev)パーサーの動作は確認できるので、 改行したい箇所があれば教えて下さい。

marihachi avatar Oct 15 '23 07:10 marihachi

if condition
  true_case
else
  false_case

if condition
{
  true_case
}

などが構文エラーになるようになりましたが、このようなスタイルの禁止は意図したものでしょうか? (私は上の括弧を使わずに改行するスタイルを使用していました。) if, for, each, while, do…while などにおいて、空白だけではなく改行で区切れても混乱を招かないのではないかと思いますが、どうでしょうか。

yuriha-chan avatar Aug 20 '25 20:08 yuriha-chan

そこは確かに改行できてもよさそうですね。修正を検討してみます

FineArchs avatar Aug 21 '25 13:08 FineArchs

【HELP】試しに作ってみたら勢い余って改行可能箇所増やしすぎた( https://github.com/FineArchs/aiscript/pull/12 )減らすのめんどい

FineArchs avatar Sep 10 '25 12:09 FineArchs

@FineArchs aiscript-dev/masterでなくフォークのmasterへのプルリクエストなのは何か意図があるのですか?

takejohn avatar Sep 10 '25 12:09 takejohn

@FineArchs aiscript-dev/masterでなくフォークのmasterへのプルリクエストなのは何か意図があるのですか?

試作品なので本家のPRとして表示させずCIだけ走らせたかったからです…

FineArchs avatar Sep 10 '25 12:09 FineArchs