Skip to content

Conversation

@t-kazu
Copy link
Collaborator

@t-kazu t-kazu commented Oct 15, 2018

  • rubyCodeの型宣言の警告
    screenshot from 2018-10-15 14-04-40
  • previewInfoVisibleの警告
    screenshot from 2018-10-15 14-05-29

@takaokouji
Copy link

確認です。

previewInfoVisible のほうは、初期表示のダイアログを消したことによる警告ですので、問題ないと思います。もし、認識に間違いがあればご指摘ください。

rubyCode のほうは、なぜでしたかね? 文字列を示す string 自体は問題なさそうなんだけど、どうして問題なのか説明できますか? 気になっているのは、rubyCodeが2回使われていることです。rubyCodeを使っている箇所が間違っていたりしないのかな。

@takaokouji
Copy link

rubyCode について、わかりました。

src/reducers/ruby-code.js によって、rubyCode.rubyCodeが定義されているからなのですね。
なるほど。
rubyCode.rubyCodeは微妙なので、rubyCode.codeがいいかな。

とりあえず、このPRは 👍 です。

@takaokouji takaokouji merged commit 18bc65e into smalruby:develop Oct 15, 2018
@t-kazu t-kazu deleted the fix/warring branch October 16, 2018 05:34
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.

2 participants