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

「非推奨だった bool 型に対するインクリメント演算子を削除」ページのコードが実行可能になっていない #69

Closed
yumetodo opened this issue Dec 4, 2017 · 16 comments
Assignees
Labels

Comments

@yumetodo
Copy link
Member

yumetodo commented Dec 4, 2017

例:

他にもあるかもしれない。

@faithandbrave
Copy link
Member

コードブロックいくつか修正しましたが、この修正と関係なく動いていなかったところもあります。
cpprefjp/site@689f3a9

yumetodo added a commit to cpprefjp/site that referenced this issue Dec 4, 2017
These codeblock is not executable!
@faithandbrave

ref:
- cpprefjp/kunai#69
@saki7
Copy link
Contributor

saki7 commented Dec 4, 2017

@yumetodo コードが実行可能にならないことが問題のはずなので、コードを実行可能にすれば問題が解決すると思うのですが、何故revertしたのですか?

@faithandbrave
Copy link
Member

あ、すいません。ぼくの修正が話をややこしくしてる気がします。
@yumetodo さんがrevertしたぼくの修正以外のところも、実行可能になっていませんでした。

@yumetodo さんがrevertしたところは、コンパイルエラーになることを意図した「削除された機能のコード」だと思います。しかし、これは方針としては「インクルードとmain関数を含む完全なコードにはexampleを付ける」というものになっていました。
Wandboxで実行することで、実際にコンパイルエラーになることを確認できるので、それも実行可能にしたほうがよいかと思います。この方針を変更したい場合は、cpprefjp/siteでIssueを立て、編集者間の合意をとってルール改定をしてください。

@saki7
Copy link
Contributor

saki7 commented Dec 4, 2017

把握しました。それで、僕の認識も @faithandbrave さんと同じなので、 @yumetodo さんは、「非推奨だった bool 型に対するインクリメント演算子を削除」のページ内にある完全なコード全てに example タグをつけて頂けますか?そうすれば、全体で統一が取れて、このIssueの問題も解決すると思います。

@yumetodo
Copy link
Member Author

yumetodo commented Dec 4, 2017

@saki7 cpprefjp/site@9c170ba をrevertするか否かはこのIssueに関係ないです。なぜかは @faithandbrave さんが

@yumetodo さんがrevertしたぼくの修正以外のところも、実行可能になっていませんでした。

と言及しているとおりです

@yumetodo
Copy link
Member Author

yumetodo commented Dec 4, 2017

cpprefjp/site_generator#30 (comment)
※念のため補足ですが、 このIssueの内容が全て実装された場合、 Kunai は md を見に行かなくなるので、 cpp example と書いたらそのまま実行可能になるわけではありません。 @melpon が に JSON を入れる実装をした時点で Kunai の挙動を切り替えます。それまでは現行の挙動を保持するつもりです

あー、えっとこれは「例」の項の中にないからまだ実行可能にならないということですかね・・・?つまり cpprefjp/site_generator#48 待ち・・・?
@saki7

@yumetodo
Copy link
Member Author

yumetodo commented Dec 4, 2017

@faithandbrave

@yumetodo さんがrevertしたところは、コンパイルエラーになることを意図した「削除された機能のコード」だと思います。

違います。変数の状態がどうなっているかわからないことに前提をおいた説明をしたいからです。

しかし、これは方針としては「インクルードとmain関数を含む完全なコードにはexampleを付ける」というものになっていました。

いつの間にそんな方針が・・・と思って見返していたら、 cpprefjp/site#473 (comment) で書かれている・・・、見落としていました。

とはいえ、これは実行可能ではないと思うので
cpprefjp/site#487
を立てました。

@saki7
Copy link
Contributor

saki7 commented Dec 4, 2017

@yumetodo:

あー、えっとこれは「例」の項の中にないからまだ実行可能にならないということですかね・・・?つまり cpprefjp/site_generator#48 待ち・・・?

#69 (comment)

これ(yumetodoさんのコメントの引用部分)はフロントエンドの実装詳細の話なので、気にしなくて良いです。完全に実装が終わるまでは現行の判別処理を引き続き使うよということです。

現行の判別処理はこれです

kunai/js/kunai/meta/meta.js

Lines 150 to 152 in 4ea3bbe

static isSampleCode(lang, info) {
return info.includes('example')
}


example を付ける場面についてはIssueが分離したので 487 の方に書きます

@saki7
Copy link
Contributor

saki7 commented Dec 5, 2017

この件はkunaiには問題はないのですが、いまさらIssueを移動するのもアレなのでこのまま続けますが、タイトルは変えます。

cpprefjp/site#488 の対応で漏れが無くなったと思いますが、 「非推奨だった bool 型に対するインクリメント演算子を削除」ページの example タグはrevertされたままなので、コンフリクトを防ぐためにここへのexampleタグへの追加は @yumetodo さんにお願いします。

ここにexampleタグを付けたい背景は cpprefjp/site#487 (comment) で書いた通りです。対称性のため、「非推奨だった bool 型に対するインクリメント演算子を削除」のコード片にも exampleタグを付けてください。

@saki7 saki7 changed the title 一部のページでコードが実行可能にならない 「非推奨だった bool 型に対するインクリメント演算子を削除」ページのコードが実行可能になっていない Dec 5, 2017
@saki7 saki7 added the bug label Dec 5, 2017
@yumetodo
Copy link
Member Author

yumetodo commented Dec 5, 2017

「非推奨だった bool 型に対するインクリメント演算子を削除」ページの example タグはrevertされたままなので

全部exampleタグが外れているわけではありません。

現状ついているものすら実行可能にならないことについて改めて原因を教えてください。

@saki7
Copy link
Contributor

saki7 commented Dec 5, 2017

あー。原因分かりました。これ(付いているのに表示されないもの)は実装のバグなのでIssue立てます

@yumetodo
Copy link
Member Author

yumetodo commented Dec 5, 2017

exampleタグへの追加は @yumetodo さんにお願いします。

現在自分の過去すべての編集を遡って確認中です。

実装のバグなのでIssue立てます

わかりました。

@saki7
Copy link
Contributor

saki7 commented Dec 5, 2017

途中「kunaiに問題ない」とか書いてたのは、単純に全てのexampleタグを削除したために丸ごと消えているものだと早とちりしていました。なので、このIssueで微妙にすれ違いみたいなのがあったのは僕が勘違いしてたからです。

yumetodo added a commit to cpprefjp/site that referenced this issue Dec 5, 2017
@yumetodo
Copy link
Member Author

yumetodo commented Dec 5, 2017

@saki7 やっぱりすれ違っていましたか。とりあえずこのIssueは cpprefjp/site@fcf8708 を持って閉じ、バグについては #71 へ。

@yumetodo yumetodo closed this as completed Dec 5, 2017
@saki7
Copy link
Contributor

saki7 commented Dec 13, 2017

#74 で直ったと思う(本番に反映済) @yumetodo

@yumetodo
Copy link
Member Author

確認しました。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants