Skip to content

news追加: add collaborator assignment of leko #259

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

Merged
merged 2 commits into from
Feb 16, 2018
Merged

news追加: add collaborator assignment of leko #259

merged 2 commits into from
Feb 16, 2018

Conversation

kt3k
Copy link
Member

@kt3k kt3k commented Feb 14, 2018

表現のチェックなどよろしくお願いします 🙇

2018-02-14 15 30 59

@watilde
Copy link
Member

watilde commented Feb 14, 2018

いいですね :)

cc @Leko

Copy link
Member

@hiroppy hiroppy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

別PRで大丈夫ですが、タイトル部分、line-heightがあったほうが重なりがなくていいかも。

@kt3k
Copy link
Member Author

kt3k commented Feb 14, 2018

@abouthiroppy AZS! 自分もそこ気になっています 😓

依存 css ライブラリの tacit の方を直すか、この repo 内で修正するか検討して修正します

Copy link
Member

@Leko Leko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

記事ありがとうございます!細かいとこですが1つ見つけましたのでコメント 👀


昨年のNode学園祭スピーカ [Leko][]さんが、Node.js の Collaborator になりました。

[Leko][]さんは昨年のNode学園祭の Code and Learn セッションで初めて Node.js にコントリビュートを開始し、その後継続的にコントリビュートを続けた結果、2018年2月に正式に Node.js の Collaborator に迎え入れられました 🎉
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR読み返して見たら、2月ではなく1月にコラボレータのPR上げてました!
nodejs/node#18117

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

なるほどです、では1月に update しますー、

実は、何をもって、就任日なのかの根拠が突き詰めるとよくわからなかったのですよね・・・

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

確かに。
私はPRがマージされた瞬間なのかな、と思ってましたが他の方の見解も聞いて見たいです

@kt3k kt3k merged commit 3a4d884 into source Feb 16, 2018
@kt3k kt3k deleted the feature/leko branch February 16, 2018 00:18
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.

4 participants