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

練習問題の一部について、メソッドを定義させるように修正 #54

Merged
merged 2 commits into from
Mar 16, 2016
Merged

Conversation

kmizu
Copy link
Contributor

@kmizu kmizu commented Mar 16, 2016

  • tut:invisibleを使って解答例を書いた分は answer.md から削除していく

@kmizu
Copy link
Contributor Author

kmizu commented Mar 16, 2016

  • 一発でビルド通るかな?

@matsu-chara
Copy link
Contributor

#29 関連 ありがとうございます(そして全然できてなくてすみません)

tut:invisibleだと読者は回答を見れないようになるのでしょうか?

@kmizu
Copy link
Contributor Author

kmizu commented Mar 16, 2016

tut:invisibleだと読者は回答を見れないようになるのでしょうか?

ドキュメント読んだ感じだと見られなくなるような…

そして、ビルドが失敗している…

@xuwei-k
Copy link
Contributor

xuwei-k commented Mar 16, 2016

tut:invisible は既に使ってる箇所あったはずなので、どうなるのか?はそれを見てもらえば

@xuwei-k
Copy link
Contributor

xuwei-k commented Mar 16, 2016

tut:invisibleを使って解答例を書いた分は answer.md から削除していく

ん?これやっていい理由がよくわからないです。答えどうやって見せるようにする予定ですか?

@xuwei-k
Copy link
Contributor

xuwei-k commented Mar 16, 2016

tut:invisibleを使って解答例 を書いても、成果物のhtmlやepubには、それらは全く含まれないはずなので、
どうしても重複が嫌で、かつ tut:invisible を使うなら
「答えはもとのgithub上のmarkdownを見ろ」
とするのは不可能ではないけど、それでは使いにくそう、という意味で

@xuwei-k
Copy link
Contributor

xuwei-k commented Mar 16, 2016

まだ失敗してる

[tut] *** Error reported at control-syntax.md:147
<console>:7: error: ';' expected but '(' found.
  } while(i < 10)(
                 ^

https://travis-ci.org/dwango/scala_text/builds/116333692#L706

@kmizu
Copy link
Contributor Author

kmizu commented Mar 16, 2016

ちゃんと手元でやるべきですね…。修正したらrebaseしときます…。

「答えはもとのgithub上のmarkdownを見ろ」

まあ、ちょっと悩んだのですが、現状markdown含めて公開されているので、markdownへのリンク貼ればいいのではという気がしてきました。

@kmizu
Copy link
Contributor Author

kmizu commented Mar 16, 2016

とりあえず、answer.md戻したうえでrebaseします…。

@xuwei-k
Copy link
Contributor

xuwei-k commented Mar 16, 2016

markdownへのリンク貼ればいいのでは

他の人が賛成なら、その方針ならそれはそれで(他にいい案ないなら)ありな気はしますけど、少なくとも回答消すのと同時にリンク貼るコミットは欲しいですね
(このpull reqに現状そういうコミットが含まれていない)

@kmizu
Copy link
Contributor Author

kmizu commented Mar 16, 2016

少なくとも回答消すのと同時にリンク貼るコミットは欲しいですね

了解です

@xuwei-k
Copy link
Contributor

xuwei-k commented Mar 16, 2016

ちゃんと手元でやるべき

手元でもいいし、こういう方法でも http://d.hatena.ne.jp/xuwei/20151205/1449294532

Kota Mizushima added 2 commits March 16, 2016 19:31
* do-whileの練習問題を改善
* mkStringの練習問題をメソッドjoinByCommaを定義するという形に改善
* tut:invisibleで解答例を入れたものはanswer.mdから順次削除(解答例は.mdへの直リンクを見に行かせる)
kmizu pushed a commit that referenced this pull request Mar 16, 2016
練習問題の一部について、メソッドを定義させるように修正
@kmizu kmizu merged commit a482ffe into scala-text:master Mar 16, 2016
@kmizu
Copy link
Contributor Author

kmizu commented Mar 16, 2016

テスト通ったのでマージ

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

3 participants