17
14

Delete article

Deleted articles cannot be recovered.

Draft of this article would be also deleted.

Are you sure you want to delete this article?

More than 3 years have passed since last update.

JavaScriptAdvent Calendar 2019

Day 10

VSCodeにコントリビュートする / VS Code Meetup #1

Posted at
1 / 26

ある日ふと思いました


OSSやろう


OSSやろう

  • React, TypeScript, Node ...
  • どちらかというと、アプリケーション寄りのOSSをやりたい

VSCode


VSCodeはOSS


けっこうアクティブ(それはそう)

スクリーンショット 2019-12-15 11.06.47.png

けっこうアクティブ

ex.react
スクリーンショット 2019-12-15 11.06.55.png


さっそくPRあげるぞ!

  • これあったらいいんじゃね?という機能を思いつきで実装し始める。

しばらくコードを読んで挫折🙃


しばらくコードを読んで挫折🙃

  • ソースコードが巨大
  • 既存の設計を理解して実装する必要がある
  • 時にその設計を修正しつつ新規でコードを追加する必要がある

ドキュメントに立ち返る


good first issue

  • good first issueというラベルがGitHubデフォルトで用意してある
  • リポジトリのニューカマーが着手しやすいissueにつけられており、他のOSSでも見かける
  • =>まずはここから着手できそうなissueを探していった

実際に取り組んだPR


help wanted

  • help wantedもGitHubデフォルトで用意してあるラベル
  • 手がつけられないけど助けてほしい、誰かやってくれない?というissueにつけられている
  • =>ここも漁ってみた

実際に取り組んだPR


リリースまで

  • 毎月イテレーションプランが決まられていて、リリースが近くなるとTestやVerificationのチェックリストが埋められてゆく
  • 無事確認されればリリースノートに名前が乗る
  • (僕もどこかに乗っています https://code.visualstudio.com/updates/v1_41)

リリースまで

スクリーンショット 2019-12-15 11.34.01.png

おまけ


CI


CI

  • CIはGitHub ActionsとAzure Pipelines
  • GitHub Actionsを本格的に使っているプロジェクトのひとつ
  • ユニットテストや結合テストをクロスOSでElectronアプリのテストをしている
  • ※Azure Pipelinesはあまり見れていない

テストカバレッジ


テストカバレッジ

全体的には40-50%くらい
スクリーンショット 2019-12-15 11.29.22.png


テストカバレッジ

テスト自体書かれていないファイルもちらほら。これで大丈夫なの?🤔
スクリーンショット 2019-12-15 11.30.29.png


More

  • より詳しいことはwikiに結構書いてある
  • ビルド、テスト、デバッグ、Coding Guidelinesなど
  • 例えばSource Code Organizationでは、全体の設計指針などが軽く書いてあったりする
  • 思いつきで気になる箇所から読み進めていくのもあり

おわり

ありがとうございましたm m

17
14
0

Register as a new user and use Qiita more conveniently

  1. You get articles that match your needs
  2. You can efficiently read back useful information
  3. You can use dark theme
What you can do with signing up
17
14

Delete article

Deleted articles cannot be recovered.

Draft of this article would be also deleted.

Are you sure you want to delete this article?