Open sotayamashita opened 9 years ago
LGTMです。
LGTMです!
:+1:
@sotayamashita あ、そうだ。PR出すまではいいとして、レビューとマージは、どうしましょうかね。
あ、3のgit branch -b feature/japaneseってなってるのは、たぶん、git checkout -b feature/japanese ですかね。いちおう。
あ、そうだ。PR出すまではいいとして、レビューとマージは、どうしましょうかね。
@tai2 そうですね。自分はオープンソースにそこまでまだ関わったことがないのでわからないのですが、マージするかどうかはコラボレーターの半数がOKみたいな感じなんですかね?
あ、3のgit branch -b feature/japaneseってなってるのは、たぶん、git checkout -b feature/japanese ですかね。いちおう。
@tai2 ありがとうございます。修正いたしました
wikiで翻訳の流れ等をwikiにまとめております。
@sotayamashita ぼくもオープンソースぜんぜん詳しくないです。仕事のやつだったら、チームメンバーが全員目を通したらOKとかだと思うんですが、こういうケースだとどうしたらいいのか、難しいですね。オープンソースでの共同作業に詳しい人の意見を待ちたい所です。
全員目を通すのは難しいと思うので、ある程度LGTMもらったらマージで良いのではないでしょうか。 誰も数日見てくれなかったらマージしちゃっても良いかもです。
翻訳文だけですし、おかしかったらまた直しましょう、でも良いと思います。
@tgfjt そうですね。何人かオープンソースに関わった人に聞いてみたのですが翻訳だけでればそこまで重大なバグとかがないので翻訳したらどんどんマージして全体のレベルを上げていけばいいと思います
誰がやるか等何も話しあえていなかったのですが、時間も迫っているので、 how-to-npm-jp は、自分の方で merge と publish しちゃいました。 これでメンターの方々もインストールしやすいと思います。
もう少し、マージなど権限を広めても良いと思っています。 @sotayamashita
お、公開されてますねー。ありがとうございます。 :+1:
@tgfjt
確かに、 @tai2 さんなどは自分よりも活発に協力してくださっているのでマージなどを権限があったほうがストレスなく翻訳ができますね。権限をadminに修正致しました。
翻訳の流れとして以下のような方法を考えたのですがどうでしょうか?
翻訳の流れ:
feature/japanese
ブランチ作成feature/japanese
をdefault branchにする例、
npm/how-to-npm
の翻訳:https://github.com/npm/how-to-npm
をforkgit remote add upstream https://github.com/npm/how-to-npm
git checkout -b feature/japanese
feature/japanese
をdefault branchにするhttps://github.com/nodeschool-jp/how-to-npm
をforknodeschool-jp/how-to-npm
にプルリクエスト