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

属性名の予約語を禁止 #863

Merged

Conversation

takejohn
Copy link
Contributor

What

属性名に予約語を使用すると文法エラーにします。

Why

Additional info (optional)

test/keywords.tsにおいて現在の文法において使用されていない予約語を追加します。
また、予約語をランダムに選択するのではなく、すべての語についてテストするようにします。
exeからparseに変えているのは気休め程度の負荷軽減……

@codecov-commenter
Copy link

codecov-commenter commented Nov 24, 2024

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

Codecov Report

All modified and coverable lines are covered by tests ✅

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

Files with missing lines Coverage Δ
src/parser/visit.ts 100.00% <100.00%> (+5.26%) ⬆️

... and 29 files with indirect coverage changes

@takejohn takejohn merged commit 5654151 into aiscript-dev:master Nov 30, 2024
4 checks passed
@takejohn takejohn deleted the fix/812_prohibit-reserved-word-in-attr branch November 30, 2024 01:53
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