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

enhance: 変数宣言で分割代入に対応 #763

Merged
merged 10 commits into from
Aug 27, 2024

Conversation

uzmoi
Copy link
Member

@uzmoi uzmoi commented Aug 24, 2024

What

タイトルの通りです。
ネームスペースでは使用できなくしました。

Why

#511

Resolve #692

Additional info (optional)

@codecov-commenter
Copy link

codecov-commenter commented Aug 24, 2024

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

Attention: Patch coverage is 73.78641% with 27 lines in your changes missing coverage. Please review.

Files Patch % Lines
src/interpreter/index.ts 68.35% 25 Missing ⚠️
src/parser/plugins/validate-keyword.ts 77.77% 2 Missing ⚠️

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Files Coverage Δ
src/node.ts 98.36% <100.00%> (ø)
src/parser/syntaxes/statements.ts 93.25% <100.00%> (ø)
src/parser/plugins/validate-keyword.ts 88.61% <77.77%> (ø)
src/interpreter/index.ts 85.80% <68.35%> (ø)

@salano-ym
Copy link
Member

エラー時のテストもあると良いと思います

@uzmoi
Copy link
Member Author

uzmoi commented Aug 25, 2024

エラー時のテストもあると良いと思います

追加しました(こういうことで合ってますかね?)

@salano-ym
Copy link
Member

changelogにネームスペースでは使用できない等の注意記載も欲しいです。
定義済み変数は使えない旨もあるとより親切かと思います。
docsの記載は別にしますか?

@uzmoi
Copy link
Member Author

uzmoi commented Aug 25, 2024

docsの記載は別にしますか?

忘れてました。
既に代入の方には分割代入の項目あるのでどんなふうに書いたらいいのか悩ましい

@salano-ym
Copy link
Member

既に代入の方には分割代入の項目あるのでどんなふうに書いたらいいのか悩ましい

既存の説明に補足する形で書くのはどうでしょう

@uzmoi
Copy link
Member Author

uzmoi commented Aug 25, 2024

既に代入の方には分割代入の項目あるのでどんなふうに書いたらいいのか悩ましい

既存の説明に補足する形で書くのはどうでしょう

宣言の構文の情報を代入の方に書くのはどうなんだっていうのが……

@salano-ym
Copy link
Member

宣言の構文の情報を代入の方に書くのはどうなんだっていうのが……

宣言の説明は先に出ているので、代入が主軸ということにして宣言と同時にできるという体にすればおかしくはないと思います。

@FineArchs
Copy link
Member

まあおかしくは無いですが、素直に宣言の方に書いた方が読みやすいような?

@salano-ym
Copy link
Member

「宣言の構文」というのはあくまで実装上の話なので説明までそれに合わせる必要は無いと思います

@uzmoi
Copy link
Member Author

uzmoi commented Aug 25, 2024

とりあえず代入の所に追加してみましたがどうでしょうか

@uzmoi uzmoi merged commit d63b8b6 into aiscript-dev:master Aug 27, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

分割代入を初期化式付き変数宣言のときにもできるようになってほしい
4 participants