-
Notifications
You must be signed in to change notification settings - Fork 13
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
Conversation
いいですね :) cc @Leko |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
別PRで大丈夫ですが、タイトル部分、line-heightがあったほうが重なりがなくていいかも。
@abouthiroppy AZS! 自分もそこ気になっています 😓 依存 css ライブラリの tacit の方を直すか、この repo 内で修正するか検討して修正します |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
記事ありがとうございます!細かいとこですが1つ見つけましたのでコメント 👀
source/news/2018/leko.md
Outdated
|
||
昨年のNode学園祭スピーカ [Leko][]さんが、Node.js の Collaborator になりました。 | ||
|
||
[Leko][]さんは昨年のNode学園祭の Code and Learn セッションで初めて Node.js にコントリビュートを開始し、その後継続的にコントリビュートを続けた結果、2018年2月に正式に Node.js の Collaborator に迎え入れられました 🎉 |
There was a problem hiding this comment.
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
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
なるほどです、では1月に update しますー、
実は、何をもって、就任日なのかの根拠が突き詰めるとよくわからなかったのですよね・・・
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
確かに。
私はPRがマージされた瞬間なのかな、と思ってましたが他の方の見解も聞いて見たいです
表現のチェックなどよろしくお願いします 🙇