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

編集者向けの細かい慣例を明文化する #481

Closed
saki7 opened this issue Nov 10, 2017 · 14 comments
Closed

編集者向けの細かい慣例を明文化する #481

saki7 opened this issue Nov 10, 2017 · 14 comments

Comments

@saki7
Copy link
Contributor

saki7 commented Nov 10, 2017

from #473

@yumetodo
Copy link
Member

yumetodo commented Dec 4, 2017

@faithandbrave 0258311 の内容だと、「例」の項の中のみにこのsynatxが適応されるように読めるのですが・・・

@yumetodo yumetodo reopened this Dec 4, 2017
@faithandbrave
Copy link
Member

雛形に全てのルールを記載するわけではないです。Issueの議論を遡ること、ほかのページとスタイルを合わせることが編集者に求められます。
3,000ページ超の情報を提供している本サイトにおいて、類似ケースを探し出せない状況は数少ないです。

分厚い辞書のようなルールを渡されて、それら全てを満たす記事を書くことは参入障壁にもなるため、明記してあるルールは最小にし、Pull Requestを受け入れる基準も甘めにし、細かいスタイルの違いはスタイルを知っている人が周りに合わせるよう修正します。
なので、歴史と周りを知ってください。

@e-kwsm
Copy link
Contributor

e-kwsm commented Dec 4, 2017

雛形に全てのルールを記載するわけではないです。

「例」以外で cpp example をつけるのは多分まれなのでこのままでいいと思います.

Pull Requestを受け入れる基準も甘めにし、細かいスタイルの違いはスタイルを知っている人が周りに合わせるよう修正します。

には同意ですが,ルールは明文化した方がいいと思います.

@faithandbrave
Copy link
Member

ルールとしてどのようなものを想像されているかわからないですが、たとえば以下のようなものは前例をルールと見なしています:

雛形は基本ルールですが、基本ルールだけで全てのケースに対応できるわけではありません。類似ケースの問題解決を事例として残し、それを見つけて同様の対応をするのは、どうしても必要となります。

@e-kwsm
Copy link
Contributor

e-kwsm commented Dec 6, 2017

たとえば以下のようなものは前例をルールと見なしています:(略)

これらを参照するよう wiki か working_style.md に書いておいた方がいいと思います.

それと, cpprefjp/site_generator#10

@melpon に以前確認したところ、タイトル(見出し1)はなにも修飾しないで記述するのが方針だそうなので

はあまり認識されていないと思うのでこれもどこかに書いた方がいいと思います.

@faithandbrave faithandbrave reopened this Dec 6, 2017
@faithandbrave
Copy link
Member

クローズされたIssueで議論を続けるのはよくないので、再オープンします。
詳細ルールを明記することを止めはしませんが、メンテナンスまでお願いします。

@saki7
Copy link
Contributor Author

saki7 commented Dec 6, 2017

@e-kwsm

サンプルコードの拡張記法の明文化について、 #487@yumetodo さんにお願いしているので、その他の明文化の作業をするのであれば、連携をお願いします。コンフリクトを避けるため、作業内容は事前に分担をお願いします。

@faithandbrave
Copy link
Member

諸々の経緯は、以下を参照していただければと思います。不明点があったら聞いてください。

@e-kwsm
Copy link
Contributor

e-kwsm commented Dec 6, 2017

@melpon HTML のコメントなどが無効化されていたと思いますが,どこを見ればいいですか?

@e-kwsm
Copy link
Contributor

e-kwsm commented Dec 6, 2017

@saki7
Copy link
Contributor Author

saki7 commented Dec 6, 2017

@e-kwsm 一部の設定情報は cpprefjp/kunai_config#1 によって分離されることが決まっているので、ここに含まれるデータはこれを参照するようにお願いします。今は、バックエンドの実装に着手していないので一時的に古いデータが分散している状況です。

sumomoneko added a commit to sumomoneko/site that referenced this issue Dec 7, 2017
@saki7
Copy link
Contributor Author

saki7 commented Dec 7, 2017

@e-kwsm:

@melpon HTML のコメントなどが無効化されていたと思いますが,どこを見ればいいですか?

おそらく、通知が流れてしまっていると思うので、僕が代わりに答えます。

調べたところ、

https://github.com/cpprefjp/markdown_to_html/blob/a567e5b66ba5523724f671a87d1ffcad550dd44d/html_attribute.py#L131

https://github.com/cpprefjp/markdown_to_html/blob/a567e5b66ba5523724f671a87d1ffcad550dd44d/html_attribute.py#L17

バックエンドではここの部分でHTMLタグを正規表現でホワイトリストしているので、ここに含まれていない要素(コメント要素など)は変換されないという挙動らしいです。

yumetodo added a commit that referenced this issue Dec 8, 2017
To notify when example tag should be used.

ref:
- #487
- #481
yumetodo added a commit that referenced this issue Dec 9, 2017
yumetodo added a commit that referenced this issue Dec 9, 2017
yumetodo added a commit that referenced this issue Dec 9, 2017
yumetodo added a commit that referenced this issue Dec 9, 2017
yumetodo added a commit that referenced this issue Dec 9, 2017
What is most important change in this Pull Request is detail information about example tag.

ref:
- #487
- #481
@saki7 saki7 changed the title サンプルコード拡張記法についての編集者向けドキュメントを整備する 編集者向けの細かい慣例を明文化する Dec 9, 2017
@saki7
Copy link
Contributor Author

saki7 commented Dec 9, 2017

今は @e-kwsm さんが作業しているので ( #481 (comment) )、assigneeに反映しておきます

@faithandbrave
Copy link
Member

こちら、 @e-kwsm さんが随時Wikiにまとめてくださっているので、ここで議論するべきことは終わったと判断し、クローズします。

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

No branches or pull requests

4 participants