che-tsumi icon indicating copy to clipboard operation
che-tsumi copied to clipboard

Separate lib folder from che-tsumi

Open re-fort opened this issue 6 years ago • 5 comments

che-tsumi-coreみたいなものを作ってche-tsumiはそれに依存する形式に変えたい

背景: che-tsumiの価値は、ある特定のリポジトリを監視して、違うリポジトリにその変更を追従できること。 それに伴うcoreの機能(lib)はche-tsumi-coreで提供する プロジェクトごとにやりたいことは違うと思うので、che-tsumiindex.jsを変更してそれを実現する che-tsumiindex.jsはあくまで雛形を提供するもの

@potato4d どう思います?

re-fort avatar Jan 31 '19 13:01 re-fort

その方針でいくならその方針で良いと思いますー! 決めの問題だと思うので。

ただ、その一方でこれを中心にしていくと、より大きなコミュニティ単位でこのコードベースに対して機能追加などの拡張ができると思っていて、折角多くの人が改善に入ってくれるかもしれない状況なので、ちょっともったいないかも?と思ったりはします

potato4d avatar Jan 31 '19 14:01 potato4d

ありがとうございます!

より大きなコミュニティ単位でこのコードベースに対して機能追加などの拡張ができると思っていて、折角多くの人が改善に入ってくれるかもしれない状況

機能拡張という意味だと、今の形だとforkした先でche-tsumiが育っていく形になっちゃうのが見えるんですよね vuejs-jp/che-tsumiのlibに機能を加えたとしてもvuejs-jp/che-tsumi/index.jsはVue用に最適化していくので、他のプロジェクトもindex.jsを最適化していくと、本家のlibの機能を入れたいけどindex.jsがconflictしてダルいみたいな形になるのかなと coreで切り出しておくと、そこにPR送ってくれて機能拡張しやすいのかなーと...認識あってますかね?

re-fort avatar Jan 31 '19 14:01 re-fort

あーなるほど、個人的にはこのレポジトリに取り込まれることによって、 fork が育つのを避けられるのかなぁと思っていたりしました( Vue に最適化されてるならこっちは触るのをやめようみたいな)。 なので、基本的には汎用的な作りにするほうが、プロジェクトとしての息は長くなるんじゃないかなぁと。

とはいえなんかこれ、割と決めの問題な気がするので、 re-fort さんが core 分割が良いと思うなら、それで良いと思います!

potato4d avatar Jan 31 '19 14:01 potato4d

意見ありがとうございます! 個人的感覚でcoreに分けさせてもらいます!!

re-fort avatar Jan 31 '19 14:01 re-fort

LGTM!

potato4d avatar Jan 31 '19 14:01 potato4d