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

strictモードが機能するよう修正 #855

Merged
merged 1 commit into from
Oct 9, 2020

Conversation

goe0
Copy link
Contributor

@goe0 goe0 commented Oct 8, 2020

🔨 変更内容 / Details of Changes

  • strictモードが機能していなかったので修正しました

🔖 関連Issue, 変更理由 / Related Issues, Reason for Changes

use strict はシングルorダブルクオートで囲う必要があるため

📷 スクリーンショット / Screenshot

📝 その他コメント / Other Comments

  • strictモードが実行されることにより一件エラーが発生しましたが、単純な変数宣言漏れだったためついでに修正しています

@codeclimate
Copy link

codeclimate bot commented Oct 8, 2020

Code Climate has analyzed commit a935e7a and detected 0 issues on this pull request.

View more on Code Climate.

Copy link
Owner

@cwtickle cwtickle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

\(^o^)/
ver2からstrictモード効いていなかったということですね;
マージOKですが、customjsやスキンjsに波及するので作品によっては注意が必要です。
十分な確認を取ってから上げてもらうよう、(特に改造している方には)注意喚起します。

ソース本体としてはすべてマージしますが、
danoni_main.jsがstrictモードになれば他のファイルは指定が無くても
strictモードになるので、ファイルの入れ替え対象として他のファイルはあえて明示しないようにします。

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.

None yet

2 participants