Skip to content
This repository has been archived by the owner on Oct 16, 2018. It is now read-only.

add channel name of slack #17

Merged
merged 1 commit into from
Sep 14, 2018

Conversation

nasa9084
Copy link
Contributor

to be kindness

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Aug 29, 2018
@k8s-ci-robot k8s-ci-robot requested review from cstoku and tnir August 29, 2018 03:06
@nasa9084
Copy link
Contributor Author

/assign @tnir

@cstoku
Copy link

cstoku commented Aug 29, 2018

kubernetes-retired/kubernetes-docs-ko#3 ではチャンネルへの直リンクにしているようですが、どちらが良さそうでしょう?

@nasa9084
Copy link
Contributor Author

直リンクにしちゃうと、逆に不親切かなと思ってます

  • [slack](slack.k8s.io) ([#kubernetes-docs-ja](チャンネルURL))とすると、リンクが複数かつ左では新規登録できる、右ではできないという形になる
  • [slack](チャンネルURL)とすると(koの形)新規登録できない

slackに登録済みの場合、チャンネル名は検索もできるので、この形が最適かと思うのですが、いかがでしょうか?

@cstoku
Copy link

cstoku commented Aug 29, 2018

なるほど、問題ないと思います!

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Aug 29, 2018
@cstoku
Copy link

cstoku commented Aug 29, 2018

/unassign
@tnir 良さそうだったら approveお願いします!

@tnir
Copy link
Contributor

tnir commented Aug 31, 2018

@nasa9084 私も @cstoku 同様、docs-koのように直リンクを貼っておくのが望ましいかと思っていましたが、いかがでしょう? (= [slack](slack.k8s.io) ([#kubernetes-docs-ja](チャンネルURL))

翻訳コントリビュータとしてKubernetes Slackに入っていない人をターゲットにする必要はないのかなと感じています。

@nasa9084
Copy link
Contributor Author

nasa9084 commented Sep 1, 2018

ドキュメントにコントリビュートするためにslackに入ってくる人もいるのでは・・・と思ってるんですが、そこは切り捨てるというか、気にしないと言うことでしょうか?

@tnir
Copy link
Contributor

tnir commented Sep 4, 2018

@nasa9084 Kubernetes Slackアカウント非保持者はk/websiteおよびその他のKubernetesプロジェクト同様に 残してある http://slack.k8s.io/ からSlackへのアクセスを得られるので、切り捨てていることにはならないと思います。

@nasa9084
Copy link
Contributor Author

nasa9084 commented Sep 4, 2018

#17 (comment)
に書いたように、わかりにくくなるように思いますし、チャンネルはKubernetes Slack内で検索が出来るので不要と思っています。

@cstoku
Copy link

cstoku commented Sep 11, 2018

折衷案感が凄いですが・・・
[Slack#kubernetes-docs-ja](チャンネルURL) ([Invite Link](http://slack.k8s.io/))
でいかがでしょう?

これが微妙なのであれば、新規の人が参入しやすく誤解されないであろう @nasa9084 の変更で良いと思います。

@tnir
Copy link
Contributor

tnir commented Sep 14, 2018

@cstoku 折衷案ありがとうございます。新たな折衷案で合意取るのも難しそうなので、このままにしようと思います。

/lgtm
/approve

@k8s-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: tnir

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Sep 14, 2018
@k8s-ci-robot k8s-ci-robot merged commit d464f29 into kubernetes-retired:master Sep 14, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm Indicates that a PR is ready to be merged. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants