-
Notifications
You must be signed in to change notification settings - Fork 162
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
feat: exampleタグの使う場面をドキュメントに追記 #491
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
✨
editors_doc/class_template_page.md
Outdated
@@ -30,6 +30,8 @@ namespace std { | |||
## 概要 | |||
(ここには、クラスの概要を記述します。必須事項です。) | |||
(サンプルコードが必要な場合は`example`タグを付け、コンパイル・実行可能にすることを検討してください。(ref: [cppref特有の拡張構文](specialized.md))) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
「概要」にサンプルコードを書くことは無いと思うので、記載の位置が変な気が
しました。で、ここじゃなくて「例」のところに書けばいいのでは?と思って
見に行くと、すでに example タグは付いてますしコメントで言及もありました。
そっちに "(ref: cppref特有の拡張構文)" を追加するだけでよかったりしませんか?
(いっしょに編集されてるほかのページもまとめての話です。)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@saki7 「例」以外のコードブロックにも example をつけるおねがいは
いっしょに追加されてる example タグの説明に書かれているので、
新しく編集する人も「cpprefjpを編集するには」→「cpprefjp特有の拡張構文」
と読み進めれば拾えるようにはなっていそうです。
「新規ページ作成者さんが必ず読むところに重ねての注意が欲しい」という
ことなら文書構造とは関係ない話なので「概要」の一部として書くのではなく
テンプレ先頭に上記文書への参照を置いたほうがいいんじゃないかと思いました。
「どこでも~」という意図を考えてもやっぱり「概要」の中で言及することは
合わないように思います。現状の「例」でのみ言及されている状態を問題とする
なら「概要」と「例」でのみ言及されている状態にしても解決とはいえなさそうで。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
「概要」にサンプルコードを書くことは無いと思うので
あります。
ex.)
- https://cpprefjp.github.io/reference/algorithm.html
- https://cpprefjp.github.io/lang/cpp17/inline_variables.html
「新規ページ作成者さんが必ず読むところに重ねての注意が欲しい」という
ことなら文書構造とは関係ない話なので「概要」の一部として書くのではなく
テンプレ先頭に上記文書への参照を置いたほうがいいんじゃないかと思いました。「概要」と「例」でのみ言及されている状態にしても解決とはいえなさそうで。
# page_title (ページのタイトルです)
これの直下に、ということですか?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
僕は現状の書き方でもOKですが、 @k-satoda さんの指摘のように記載場所に違和感があるということも少し分かります。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@yumetodo なるほど、概要を示すにあたって多少長くなってでも
コードを示したほうが手っ取り早いことはありますね。気づいてませんでした。
「テンプレ先頭に」はタイトルよりも上のつもりでした。タイトルの書き方に
関する情報が参照先の文書に載っててもおかしくはないと思いますし。
ただし、機械処理とか、なんか支障が出るようなら下でもいいと思います。
@yumetodo コミットコメントなどは好きに書いて良いと思うのですが、プルリクエストが英語だと少し分かりづらいので、日本語に改題していただけると助かります。 |
follow the review by @k-satoda #491 (comment) ref: - #487 - #481
@k-satoda これでどうでしょうか? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ここに移動するのであれば、意図を明確にするために、補足を入れた方が良いと思います。
- 「サンプルコードは文章中のどの部分で書いても大丈夫です。閲覧者の理解を助けるために必要だと感じたところで入れてください。」
editors_doc/class_template_page.md
Outdated
@@ -1,3 +1,5 @@ | |||
(サンプルコードが必要な場合は`example`タグを付け、コンパイル・実行可能にすることを検討してください。(ref: [cppref特有の拡張構文](specialized.md))) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
元コメントの「テンプレ先頭に上記文書への参照を~」で指していた文書は
「cpprefjpを編集するには」「cpprefjp特有の拡張構文」のことでした。
後者は前者からリンクされているので後者は無くてもいいかもしれないとも
思います。
おっと、読み間違えていました。
@saki7 さんの指摘も踏まえ反映します。
「スタイル」など他にもいろいろある全体的な編集のしかたのうち
example タグについてだけが各テンプレ先頭に置いてあるというのはちょっと
変な気がします。
やっぱりそうですよね・・・
revertします。
This reverts commit 3c324fb. #491 (comment) ref: - #487 - #481
根本的に書き直しました。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
大筋ではこれでも良いと思います。
が、途中をrevertしたことで繋がりがなくなってしまっているので直す必要があると思います(ということに今気づきました)。
以下の文章は、自己言及している気がするので、少し削っても良いんじゃないかなと思います。
その際拡張構文であるexampleタグをルールに従って付け、コンパイル・実行可能にすることを検討してください。
↓
その際は拡張構文のexampleタグを付け、コンパイル・実行可能にすることを検討してください。
exampleタグについての説明をrevertしてセクションが丸ごと消えたので、このprの現状ではexampleタグについて説明している箇所はここしかないです。
そうなると、この文章自体がルールになると思います。そこで、「ルールに従って……」と書いてしまうと「ルールってこれじゃないの?」と自己言及している風に感じると思います。
なので、exampleタグについてのrevertはrevertしない方が良いと思います。
雛形で端的に書くのは重要ですが、少し詳しく解説しているページがあっても良いと思います。ここは、僕は初稿の書き方が好きでした (Approveしてます)
あべし、それが消えたのは意図するところではなかったです・・・戻します(commit log ごちゃごちゃ)
削ると拡張構文の説明ページ(現在lost中)を見てもらえない気がします。 |
真上にリンクがあるのでそこを見てもらえるのではないですか。ここは任せます。 |
戻しました。 個人的には「ルールに従って」の部分は削るつもりはないですが他のレビューを待ちます。 |
@k-satoda さんないし @faithandbrave さんにmergeは委ねたいです。12/16 0:00 JST時点でmergeされていない場合は私がmergeします。 |
大本の原因としては、タイトルより先頭にテキストがあったこと。 これによって、 1. タイトルと本文のsplitが正常にできず、タイトルがnullを返す 2. crsearch.json が null を含んだ JSON を生成する 3. kunai が使ってる crsearch 内で文字列を期待する場所で null が渡ってエラーになる 4. 結果としてサイドバーが表示されなくなる となっていた。
#487 より。
example
タグの使う場面をドキュメントに追記ref: