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

add src/pkg/go/build/doc.go #54 #57

Merged
merged 3 commits into from
Jun 15, 2014
Merged

add src/pkg/go/build/doc.go #54 #57

merged 3 commits into from
Jun 15, 2014

Conversation

ucho
Copy link
Contributor

@ucho ucho commented Jun 14, 2014

#54

見出しを日本語で書くと見出しにならない(#15)ようなので、
ひとまず見出しは原文のままにしました。

@atotto
Copy link
Member

atotto commented Jun 14, 2014

#15 の件ですね。こちらの件はまだ手を入れられてません。。

review:

細かいところですみませんが、よろしくお願いします。

@ucho
Copy link
Contributor Author

ucho commented Jun 15, 2014

ご指摘ありがとうございます。修正しました。

改行についてですが、1行が80文字を超えてもかまわないということでしょうか?
どうもオリジナルは80文字ルールでやっているようだったので…。

もし1行の文字数を制限しないのであれば、なるべく改行を入れない方がよいかもしれません。
改行はgodocでスペースに変換されているように見えるので、なんか気持ち悪いです…。

@atotto
Copy link
Member

atotto commented Jun 15, 2014

修正ありがとうございます!

html上の改行はスペースになってしまいますので、適当な場所で改行すればいいかなと思ってます。
1行にまとめていただいても構わないのですが、長いと差分が見づらい等々・・・という感じです。

uchoさんがよければマージしちゃいます。

@ucho
Copy link
Contributor Author

ucho commented Jun 15, 2014

はい、マージしていただいてOKです。

atotto added a commit that referenced this pull request Jun 15, 2014
src/pkg/go/build/doc.go: add translate 

refs #54 #57
@atotto atotto merged commit 734cc88 into gophersjp:master Jun 15, 2014
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