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

Separate lib folder from che-tsumi #25

Open
re-fort opened this issue Jan 31, 2019 · 5 comments
Open

Separate lib folder from che-tsumi #25

re-fort opened this issue Jan 31, 2019 · 5 comments

Comments

@re-fort
Copy link
Member

re-fort commented Jan 31, 2019

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

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

@potato4d どう思います?

@potato4d
Copy link
Member

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

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

@re-fort
Copy link
Member Author

re-fort commented Jan 31, 2019

ありがとうございます!

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

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

@potato4d
Copy link
Member

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

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

@re-fort
Copy link
Member Author

re-fort commented Jan 31, 2019

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

@potato4d
Copy link
Member

LGTM!

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

2 participants