回答編集履歴
3
自分のやり方として紹介
answer
CHANGED
|
@@ -9,12 +9,17 @@
|
|
|
9
9
|
> 2. 前者の場合、バージョン番号は1.0.0に戻してしまってもいいのか
|
|
10
10
|
|
|
11
11
|
パッケージ名は変えて良いです。
|
|
12
|
-
むしろREADMEの上の方にもっとejs3対応なんだぜ感を前面に押し出した方が良い
|
|
12
|
+
むしろREADMEの上の方にもっとejs3対応なんだぜ感を前面に押し出した方が良いので
|
|
13
|
+
私も「ejs3-html-loader」みたいな名前にしてるでしょう。
|
|
13
14
|
|
|
14
|
-
もしライブラリの保守対応とかしたくない
|
|
15
|
+
もしライブラリの保守対応とかしたくないならば改名は無しが無難ですかね。
|
|
15
|
-
作者に頑張って欲しいので「こちらのプロジェクトをフォークして
|
|
16
|
+
作者に頑張って欲しいので「こちらのプロジェクトをフォークしてコミット履歴を足しました」と載せ、
|
|
16
|
-
あくまで暫定的なライブラリだということを強調
|
|
17
|
+
あくまで暫定的なライブラリだということを強調しつつサボる方向に全振りします。
|
|
17
18
|
|
|
19
|
+
npmのサイトにパブリッシュすると面倒な事になるのは目に見えているので、
|
|
20
|
+
README.mdのインストールの項目では`npm install bananacoffee/ejs3-html-loader`での導入を紹介するように記述を変えて凌ぐ形にします。
|
|
21
|
+
参考記事: [Githubまたはローカルのnpm のパッケージをinstallする方法 - Qiita](https://qiita.com/pure-adachi/items/ba82b03dba3ebabc6312)
|
|
22
|
+
|
|
18
23
|
> 3. ライセンス情報に開発者の名前があるが、差し替えるべきなのか、併記するべきなのか
|
|
19
24
|
> 4. READMEに必ず記載すべきものはあるのか
|
|
20
25
|
> 5. 3, 4に記載る名前はハンドルネーム（baananacoffee）でも良いのか
|
|
@@ -23,16 +28,21 @@
|
|
|
23
28
|
適切に取り扱ってください。
|
|
24
29
|
とはいえ、MITライセンスは世界一雑なライセンスなので、最低限のとこさえ守っていればライセンス違反にはならないでしょう。
|
|
25
30
|
|
|
31
|
+
なので取りうる手段は先程の例と同じで、
|
|
26
|
-
|
|
32
|
+
自分はライブラリのアップデートに対して責任を持つのか持たないかが重要です。
|
|
27
|
-
|
|
33
|
+
それにより振る舞いを変えていきます。
|
|
28
34
|
|
|
29
|
-
責任を持つなら自分の名前で記載しなおし
|
|
35
|
+
責任を持つなら自分の名前で記載しなおし！
|
|
30
36
|
READMEも前のプロジェクトの事はインスパイア元として一言紹介に留めておき、
|
|
31
|
-
ライブラリを見たユーザーが扱える最善の説明書きが記載されているべきです
|
|
37
|
+
ライブラリを見たユーザーが扱える最善の説明書きが記載されているべきですからね。
|
|
32
38
|
|
|
33
|
-
|
|
39
|
+
そもそもテストコードがCoffeeScriptってなんだよES6以降のコードでよくね？
|
|
34
|
-
と
|
|
40
|
+
スパッと捨てちゃってES6で書き直そう。
|
|
35
41
|
|
|
42
|
+
責任を持ちたくないコースで行くなら、あくまで暫定！
|
|
43
|
+
フォークして手直ししただけなんだよ感を前面に押し出す感じにすれば良いでしょう。
|
|
44
|
+
となればライセンスはいじらず、READMEもちょろっと変えた程度で済ませます。
|
|
45
|
+
|
|
36
46
|
---
|
|
37
47
|
|
|
38
48
|
プルリクエストを読みましたが、
|
2
もう少し文章を改良
answer
CHANGED
|
@@ -47,16 +47,21 @@
|
|
|
47
47
|
上記の事を無視したプルリクエストが来ても無視(放置)しちゃうかもしれません。
|
|
48
48
|
|
|
49
49
|
一番ツッコみたい箇所として
|
|
50
|
-
ejs3のcompatible(互換性)を謳っていますが
|
|
50
|
+
ejs3のcompatible(互換性)を謳っていますがオブジェクトのキーを勝手に削ってますよね？
|
|
51
51
|
もしかして、ejs2では動作せずejs3専用なのではないですか？
|
|
52
52
|
|
|
53
|
+
(この辺私は全然ejsには詳しくなく、試してもないので的外れだったらごめんなさい)
|
|
54
|
+
(まぁ、depsってなんやねんいらねえよと思う気持ちもわかる気がする)
|
|
55
|
+
|
|
53
|
-
ちゃんとejs2でも動作確認して
|
|
56
|
+
ちゃんとejs2でも動作確認して互換性があるなら
|
|
54
|
-
|
|
57
|
+
プルリクエストの説明書きに真っ先に記載して、
|
|
55
58
|
安心させて欲しいなと思ってしまいます。
|
|
56
59
|
|
|
57
|
-
また動作が変わりますが、テストコードは動
|
|
60
|
+
また動作が変わりますが、テストコードは動きますか？
|
|
61
|
+
コレにより失敗するテストがあるなら、テストコードを修正するのもプルリクエスト作成者の義務です。
|
|
58
62
|
`npm run test`コマンドを実行してテストも走らせてみてください。
|
|
59
63
|
その上でやはり説明書きにテスト結果に関しても軽く触れて欲しいですね。
|
|
60
64
|
|
|
61
|
-
ejs2とejs3を組み込んだテストコードも用意してテストが通る事を確認しています。
|
|
65
|
+
ejs2とejs3を組み込んだテストコードも用意してテストが通る事を確認しています、みたいな。
|
|
62
|
-
|
|
66
|
+
ここまで完璧にやれていれば、質問者さんに全く落ち度はないと思います。
|
|
67
|
+
作者が蒸発したので、npmで新しいライブラリ名で公開するかという感じになりますね。
|
1
プルリクエストに関しての記述を変更
answer
CHANGED
|
@@ -36,7 +36,7 @@
|
|
|
36
36
|
---
|
|
37
37
|
|
|
38
38
|
プルリクエストを読みましたが、
|
|
39
|
-
微妙と感じる箇所が
|
|
39
|
+
微妙と感じる箇所がいくつかありました。
|
|
40
40
|
|
|
41
41
|
- プルリクエストの説明書きが空なんだが……
|
|
42
42
|
- メソッド勝手に減らしてるけど動作は？
|
|
@@ -46,9 +46,17 @@
|
|
|
46
46
|
私がライブラリの開発者だとして、
|
|
47
47
|
上記の事を無視したプルリクエストが来ても無視(放置)しちゃうかもしれません。
|
|
48
48
|
|
|
49
|
-
|
|
49
|
+
一番ツッコみたい箇所として
|
|
50
|
+
ejs3のcompatible(互換性)を謳っていますがメソッド削ってますよね？
|
|
50
|
-
|
|
51
|
+
もしかして、ejs2では動作せずejs3専用なのではないですか？
|
|
51
|
-
まず説明書きを頑張って作ってはどうでしょう？
|
|
52
52
|
|
|
53
|
+
ちゃんとejs2でも動作確認して
|
|
54
|
+
互換性があるというならば、プルリクエストの説明書きに真っ先に記載して、
|
|
55
|
+
安心させて欲しいなと思ってしまいます。
|
|
56
|
+
|
|
57
|
+
また動作が変わりますが、テストコードは動くんですか？
|
|
53
|
-
|
|
58
|
+
`npm run test`コマンドを実行してテストも走らせてみてください。
|
|
54
|
-
その上で
|
|
59
|
+
その上でやはり説明書きにテスト結果に関しても軽く触れて欲しいですね。
|
|
60
|
+
|
|
61
|
+
ejs2とejs3を組み込んだテストコードも用意してテストが通る事を確認しています。
|
|
62
|
+
みたいな……
|