Skip to content

fix: #243 Syntax error if Made block argument name include space #255

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

Merged
merged 1 commit into from
Aug 20, 2020

Conversation

matsudai
Copy link

Resolves

Proposed Changes

MyBlockのコードからRubyの変換について以下を修正しました。

  • 引数にスペース(または引数の名前として解釈できない文字)が含まれる場合、その代わりにアンダースコアを使うようにしました
  • 引数の先頭の文字が大文字である場合、その文字を小文字に変換するようにしました

また、argsへの代入を行わなくなったためargsをletではなくconstで宣言します。

Reason for Changes

refs #243

Browser Coverage

Check the OS/browser combinations tested (At least 2)

Mac

  • Chrome
  • Firefox
  • Safari

Windows

  • Chrome
  • Firefox
  • Edge
  • New Edge

Chromebook

  • Chrome

iPad

  • Safari

Android Tablet

  • Chrome

…e space

MyBlockのコード <-> Rubyの変換について以下を修正しました。
- 引数にスペースが含まれる場合、コードへの変換に失敗すること
- 引数の先頭の文字が大文字である場合、コードへの変換に失敗すること

また、argsへの代入を行わなくなったためargsをletではなくconstで宣言します。
@takaokouji
Copy link

LGTM

@takaokouji takaokouji merged commit f7eafbc into smalruby:develop Aug 20, 2020
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.

Syntax error if Made block argument name include space
2 participants