Skip to content
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

OSS Gate Workshop: online: 2023-09-30: odentakashi Todo-Tree: Work log #1671

Closed
OdenTakashi opened this issue Sep 30, 2023 · 41 comments
Closed

Comments

@OdenTakashi
Copy link
Contributor

This is a work log of a "OSS Gate workshop".
"OSS Gate workshop" is an activity to increase OSS developers.
Here's been discussed in Japanese. Thanks.

作業ログ作成時の説明

以下のテンプレートを埋めてタイトルに設定します。埋め方例はスクロールすると見えてきます。

OSS Gate Workshop: ${LOCATION}: ${YEAR}-${MONTH}-${DAY}: ${ACCOUNT_NAME}: ${OSS_NAME}: Work log

タイトル例↓:

OSS Gate Workshop: Tokyo: 2017-01-16: kou: Rabbit: Work log

OSS Gateワークショップ関連情報

  • スライド:ワークショップの進行に使っているスライドがあります。
  • チャット:OSS開発に関することならなんでも相談できます。ワークショップが終わった後もオンラインで相談しながら継続的にOSSの開発に参加しましょう!
  • シナリオ:ワークショップの目的・内容・進め方の詳細が書いています。
  • 過去のビギナーの作業ログ:他の人の作業ログから学べることがいろいろあるはずです。
@OdenTakashi
Copy link
Contributor Author

vscodeの拡張機能から探してみる

@OdenTakashi
Copy link
Contributor Author

Todo-Treeというvscodeの拡張機能を試してみる

@miyanew
Copy link
Contributor

miyanew commented Sep 30, 2023

ライセンスは MIT License とのこと

https://github.com/Gruntfuggly/todo-tree/blob/master/License.txt

@miyanew
Copy link
Contributor

miyanew commented Sep 30, 2023

https://opensource.org/licenses/
で確認しました。OSSライセンスです。

@OdenTakashi
Copy link
Contributor Author

GitHubのリポジトリ: https://github.com/Gruntfuggly/todo-tree

@OdenTakashi
Copy link
Contributor Author

https://github.com/Gruntfuggly/todo-treeのREADME記載のインストール手順を参考にする

@OdenTakashi
Copy link
Contributor Author

OdenTakashi commented Sep 30, 2023

環境: macOS

@miyanew
Copy link
Contributor

miyanew commented Sep 30, 2023

インストール手順はこれでよさそうですね

https://github.com/Gruntfuggly/todo-tree/blob/master/README.md#installing

@OdenTakashi
Copy link
Contributor Author

vscode上で

  • cmd+p
  • ext install Gruntfuggly.todo-tree
    を実行。

@OdenTakashi
Copy link
Contributor Author

問題なくインストールできた

@OdenTakashi
Copy link
Contributor Author

使い方を探しているが見当たらない

@miyanew
Copy link
Contributor

miyanew commented Sep 30, 2023

うーんそうみえますね。
初めてみた人が使い方を見つけられるような導線が合った方が良いみたいなフィードバックできそうかも

@OdenTakashi
Copy link
Contributor Author

OdenTakashi commented Sep 30, 2023

このわからなさの背景には一般的なTODOの認識とここで使われているソースコードに直接TODOを書くというTODOへの認識の違いがあるのかもしれない 

@OdenTakashi
Copy link
Contributor Author

そもそもTODOの説明が足らないかもしれない

@OdenTakashi
Copy link
Contributor Author

ここで使われているTODOの説明: https://www.jetbrains.com/help/idea/using-todo.html

@miyanew
Copy link
Contributor

miyanew commented Sep 30, 2023

Piroさんからアドバイスいただいた
Readmeがハイコンテクストすぎないか?とのこと

Readmeの前提条件(文脈?)として
プロダクト開発(OSS開発ふくめ)においてソースコードに "TODO" などのコメントを埋め込む文化がある
ということを示すとよいかもしれない

参照リンク先
https://www.jetbrains.com/help/idea/using-todo.html
https://medium.com/imdoneio/the-imdone-pitch-feedback-welcome-386430accf01

@miyanew
Copy link
Contributor

miyanew commented Sep 30, 2023

修正方法: 外部サイトにリンクを貼るか、Readmeに文章を書くか -> 一長一短

  • リンクを貼る: メンテコストは低い、リンク先の外部サイトがいつまであるか謎
  • 文章を書く: メンテコストが高い、プロダクト内で完結できる

Github内のドキュメントを参照するならパーマリンクを貼ればいいんだけど

アメリカだとフェアユース - Wikipediaという基準がある
引用するにあたり、元記事を明らかにすれば問題にならないハズ

@OdenTakashi OdenTakashi changed the title OSS Gate Workshop: online: 2023-09-30: odentakashi ${OSS_NAME}: Work log OSS Gate Workshop: online: 2023-09-30: odentakashi Todo-Tree: Work log Sep 30, 2023
@OdenTakashi
Copy link
Contributor Author

初めて使おうとした場合に使い方がわからないと困るので報告したい。

@OdenTakashi
Copy link
Contributor Author

報告方法に指定はなさそう

@OdenTakashi
Copy link
Contributor Author

使い方がわからないという問題なのでHow to useというキーワードでissueを検索してみたが見当たらなかった

@OdenTakashi
Copy link
Contributor Author

報告テンプレートも特になし。

@OdenTakashi
Copy link
Contributor Author

伝えたいこと

  • ソースコードに "TODO" などのコメントを埋め込む文化があるという前提を知らない状態でこのREADMEを読むとここで書かれているTODOの意味が分かりずらいため、前置きとしてこのような文化があるということを明示するといいかも

@OdenTakashi
Copy link
Contributor Author

OdenTakashi commented Sep 30, 2023

READMEの修正に関しては、具体的に改善があるならば「ここをこういうふうに直すといいと思います。」のように具体案を提示するのが良い。
改善案が決まっていないならば、「ここが分かりずらいのでこういうニュアンスを加えるといいのではないか」と提案するのが良いとアドバイスいただきました。

READMEの修正は上記のものが期待値を含んでいるので別途期待値を書く必要はない

@OdenTakashi
Copy link
Contributor Author

OdenTakashi commented Sep 30, 2023

タイトル:"How about adding an explanation about TODO in the README?"

本文:
Thankyou for great extension !

But It took me some time to understand how to use it. Because I didn't know the culture of embedding comments like TODO or FIX in source code.

So, if possible, explaining such a culture might make it more understandable.

@goruchanchan
Copy link
Contributor

goruchanchan commented Sep 30, 2023

- This extension quickly searches (using [ripgrep](https://github.com/BurntSushi/ripgrep)) your workspace for comment tags like TODO and FIXME, and displays them in a tree view in the activity bar.
+ This extension quickly searches (using [ripgrep](https://github.com/BurntSushi/ripgrep)) your workspace for comment tags like TODO and FIXME, and displays them in a tree view in the activity bar.hogehoge
image

@OdenTakashi
Copy link
Contributor Author

Most programmers put TODO comments in their code to track problems they see and ideas they have, like a carpenter building a house who writes a measurement on the framing. Programmers use TODO comments because they’re convenient and allow you to stay in the code, reducing distractions.
Using TODO Comments to Keep You Programming in the Zone | by Jesse Piascik | imdone.io | Medium

@goruchanchan
Copy link
Contributor

goruchanchan commented Sep 30, 2023

アノテーションコード(コマンド?)は、わかりやすい表記にしてもいいかもです!
上に貼ってくれたリンク先の記載も他との違いがわかるようになっている気がするので👀

@OdenTakashi
Copy link
Contributor Author

OdenTakashi commented Sep 30, 2023

Thank you for great extension !

But It took me some time to understand how to use it. Because I didn't know the culture of embedding comments like TODO or FIX in source code.

So, explaining such a culture might make it more understandable.

How about adding links to explanations about the culture regarding TODO or FIXME as follows?

README.md
- This extension quickly searches (using [ripgrep](https://github.com/BurntSushi/ripgrep)) your workspace for comment tags like TODO and FIXME, and displays them in a tree view in the activity bar. The view can be dragged out of the activity bar into the explorer pane (or anywhere else you would prefer it to be).
+ This extension quickly searches (using [ripgrep](https://github.com/BurntSushi/ripgrep)) your workspace for comment tags like [TODO](https://www.jetbrains.com/help/idea/using-todo.html) and [FIXME](https://www.jetbrains.com/help/idea/using-todo.html) and displays them in a tree view in the activity bar. The view can be dragged out of the activity bar into the explorer pane (or anywhere else you would prefer it to be).

@OdenTakashi
Copy link
Contributor Author

リンクを追加する提案にしてみました。

@OdenTakashi
Copy link
Contributor Author

I didn't know というよりもより多くの人に知ってもらえるというニュアンスにすると良い。

@goruchanchan
Copy link
Contributor

https://en.wikipedia.org/wiki/Comment_(computer_programming)#Tags

@OdenTakashi
Copy link
Contributor Author

リンク切れのある可能性があるためwikiなどのリンク切れがないであろうリンクを使うようにする。

@github-actions
Copy link

おつかれさまでした!

ワークショップの終了にともないissueを閉じますが、このまま作業メモとして使っても構いません 👌

ワークショップの感想を集めています!

ブログなどに書かれた際は、このページへリンクの追加をお願いします 🙏

またの参加をお待ちしています!

@komagata
Copy link
Contributor

komagata commented Oct 4, 2023

@OdenTakashi こちらその後いかがでしょうか〜?
僕でもよろしければIssue文章のサポートなどもさせていただきます〜!

@OdenTakashi
Copy link
Contributor Author

OdenTakashi commented Oct 4, 2023

@komagata
気にかけてくださりありがとうございます🙏

今週末を目処に作業を行う予定です。

その際に文章のサポートを依頼させていただくかもしれませんが、その際はどうぞよろしくお願いします🙇‍♂️。

@OdenTakashi
Copy link
Contributor Author

メモ

PR: Add link to explanations about the culture regarding tags by OdenTakashi · Pull Request #796 · Gruntfuggly/todo-tree

@OdenTakashi
Copy link
Contributor Author

OdenTakashi commented Oct 8, 2023

@komagata
無事PRを送ることができました。
対応はこれからになると思いますが、取り急ぎ報告させていただきます。🙏

@komagata
Copy link
Contributor

@OdenTakashi PR作成おめでとうございます〜!

@komagata
Copy link
Contributor

@OdenTakashi こちら、メインのコントリビューターにメンションで、「こちらいかがでしょうか〜」的なメッセージ送ってもいいかもですね。大分日にちが経っているので。

@OdenTakashi
Copy link
Contributor Author

@komagata
アドバイスいただきありがとうございます。🙏

コメントを送りました。
Add link to explanations about the culture regarding tags by OdenTakashi · Pull Request #796 · Gruntfuggly/todo-tree

@komagata
Copy link
Contributor

@OdenTakashi スバラシィ〜

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

No branches or pull requests

4 participants