[frontend] Scratch3とテキストファイルを添付可能に

github-actions[bot] commented 1 year ago

Visit the preview URL for this PR (updated for commit 9917138):

https://szpp-judge-3776--pr118-feat-frontend-contes-jmygm841.web.app

_{(expires Wed, 11 Oct 2023 02:17:22 GMT)}

_{🔥 via Firebase Hosting GitHub Action 🌎}

_{Sign: 1f8b9d12a855b0fe650d3231cd716db46d79c069}

h-takeyeah commented 1 year ago

PRお疲れ様です！

事後でアレですが es5 → es2020 の理由をこの PR に書いといてください:bow:

それからおまけですが Error の cause は Chrome の場合は仕様で console.error に出ません．

一方，Firefox だと console.error(e) するだけで HogeError Caused by FugaError のように，そのエラーと cause を一緒に出してくれるので捗ります．なのでデバッグの目的でスタックトレースとして cause を使うのは Chrome の場合は望ましくないと思います．もちろん console.error(e.cause) のように書けば取れますが，ちょっと面倒だよねというくらいの意味です．

といってもこの PR の実装だとメッセージがかなり親切なので誤解はなく大丈夫だと思います．

arumakan1727 commented 1 year ago

チェックありがとうございます！ Chrome だと Error の cause は出ないのですね、初めて知りました :bow:

es2020 にした意味は結論からいうとないです :bow:

経緯：

class Sb3ToCppConverter を TypeScript で書き直そうとしたときに、以前たかはしさんが言ってたプライベートクラス機能 (#) を JS トランパイル後も使えるようにしたい

es2015 でも良かったが、もう少し新しくても良いだろうと思って 2020 にした

けど途中で TS 化に挫折した

結果、tsconfig の修正だけが残ったが、そのままでも問題ないだろうと思ってコミットした :bow:

szpp-dev-team / szpp-judge

[frontend] Scratch3とテキストファイルを添付可能に #118