ts-pegjsの導入 - Githubissues

sounisi5011 commented 5 years ago

https://github.com/sounisi5011/vec-draw/pull/16#issuecomment-487127021 で触れられていたts-pegjsを導入し、PEG.jsのパーサもTypeScript化する。

xパッケージを導入
[x] TypeScript向けのパーサが生成されるようにnpm scriptsを修正
[x] TypeScript向けのパーサが生成されるように./src/parser/dsl.pegjsを変更
[x] 正しい型定義が生成されるようにreturnTypesオプションを指定 ( https://github.com/sounisi5011/vec-draw/pull/16#issuecomment-487261485 )
[x] パーサのエラーを外部のError継承オブジェクトをthrowする方法に変更し、expected ()関数の引数には文字列のみを渡す ( https://github.com/sounisi5011/vec-draw/pull/16#issuecomment-487261427 )

sounisi5011 commented 5 years ago

一通りの作業は完了した。他にやることは無いだろうか？

sounisi5011 commented 5 years ago

一通りの作業は完了した。他にやることは無いだろうか？

build:pegjs:*系のnpm scriptsを消し忘れていない？

sounisi5011 commented 5 years ago

./src/parser/dsl.pegjs.d.tsはもう使わないハズ。消してしまっても良いのでは。

sounisi5011 commented 5 years ago

https://github.com/sounisi5011/vec-draw/issues/17 で提案されていた関数の外部化はどうしようか？

sounisi5011 commented 5 years ago

17 で提案されていた関数の外部化はどうしようか？

それは「ts-pegjsの導入」の対応範囲外だ。

エラークラスを読み込んだのは、expected()関数の型エラーを回避するため。それ以外の点に関しては、必要な修正とは思えない。

sounisi5011 commented 5 years ago

完了だ。closeしよう

sounisi5011 / vec-draw