CHANGELOG.md を追加する #813

takke · 2019-03-25T03:12:37Z

3/27 にとりあえずリリースということなのでせめて CHANGELOG.md くらいは用意しないと誰も評価できないと思うので追加しました。

作成方法は CHANGELOG.mdについて · sakura-editor/sakura Wiki の wiki に書いてあるとおりです。

バージョン番号部分は自動生成だと Unreleased ですがかっこ悪いので v2.4.0.0 alpha1 にしました。決まり次第変えましょう。

m-tmatma · 2019-03-25T03:37:07Z

ありがとうございます。

Appveyor で生成したままのものをそのままコミットしていただいて、
その後のコミットで、調整した方がいいです。
そうでないと、どの部分を手動修正したか追跡しにくいです。

またappeyor のどのビルドかという情報もコミットログに欲しいです。
これも後で追跡しやすくするためです。

※ rebase で修正頂きたいです。

…editor/changelog-sakura/builds/23319217

takke · 2019-03-25T04:05:10Z

Appveyor で生成したままのものをそのままコミットしていただいて、
その後のコミットで、調整した方がいいです。
そうでないと、どの部分を手動修正したか追跡しにくいです。

確かにそうですね。変更しました。

またappeyor のどのビルドかという情報もコミットログに欲しいです。
これも後で追跡しやすくするためです。

コミットログに appveyor の URL を含めました。

KENCHjp · 2019-03-25T06:50:12Z

これのコミットに合わせて、タグ(v2.4.0.0 alpha1)付け？

KENCHjp · 2019-03-25T07:04:27Z

タグに規則があるみたいっすね。

v2.4.0.0-alpha1

これなら通る（スペースが通らないのかな）
末尾がビルド番号ってことだと、

v2.4.0-alpha1

こうっすかね。

m-tmatma · 2019-03-25T13:24:48Z

v2.4.0.0-alpha1

これなら通る（スペースが通らないのかな）
末尾がビルド番号ってことだと、

v2.4.0-alpha1

手作業の修正をしなくてもいいように以下の PR を作ってみましたが、
sakura-editor/changelog-sakura#22

なんか期待と違う。

takke · 2019-03-26T07:09:09Z

v2.4.0-alpha1 のタグを下記コマンドで打って、そのタグを参照するように CHANGELOG.md を修正しました。

$ git tag v2.4.0-alpha1
$ git push upstream v2.4.0-alpha1

beru · 2019-03-26T13:27:50Z

CHANGELOG.md

@@ -0,0 +1,236 @@
+# Change Log


Change Logs もしくは Changelog が良いと思います。

何か理由はありますか？

あまり例が見つかりませんがJavaの世界で最近よく使われているOkHttpというライブラリは Change Log と表記していますし個人的に違和感はないです。

https://github.com/square/okhttp/blob/master/CHANGELOG.md

きのこたけのこ戦争みたいなものだと思います。

では Change Log でいいですかね。

beru · 2019-03-26T13:31:48Z

CHANGELOG.md

+
+- Window テキストをCNativeT で取得/設定するユーティリティ関数を追加 [\#776](https://github.com/sakura-editor/sakura/pull/776) ([m-tmatma](https://github.com/m-tmatma))
+- 開いているファイルのフォルダのパスをクリップボードにコピーできるようにする [\#718](https://github.com/sakura-editor/sakura/pull/718) ([m-tmatma](https://github.com/m-tmatma))
+- Common File Dialog の代わりに Common Item Dialog を使用する [\#716](https://github.com/sakura-editor/sakura/pull/716) ([beru](https://github.com/beru))


Issueのタイトルから生成していると思うんですが、Common File Dialog の代わりに Common Item Dialog を使用する というのは正確な記載では無いですね。Vistaスタイルのファイルダイアログを使えるようにする が正しい説明ですね。Issue のタイトルを変えてから生成しなおせば直せそうでしょうか？

Issue (というか PR ですよね？) のタイトルから自動生成しているのでそちらを直していただければ。

Issue #716 のタイトルを更新しました。

ありがとうございます！
そしてこのPRは個人的に新バージョンの目玉機能の１つだと感じているので分かりやすくなって嬉しいです。

Issue ではなくて PR でした。

CHANGELOG.mdを生成し直して修正しました。

beru · 2019-03-26T13:32:57Z

CHANGELOG.md

+- Window テキストをCNativeT で取得/設定するユーティリティ関数を追加 [\#776](https://github.com/sakura-editor/sakura/pull/776) ([m-tmatma](https://github.com/m-tmatma))
+- 開いているファイルのフォルダのパスをクリップボードにコピーできるようにする [\#718](https://github.com/sakura-editor/sakura/pull/718) ([m-tmatma](https://github.com/m-tmatma))
+- Common File Dialog の代わりに Common Item Dialog を使用する [\#716](https://github.com/sakura-editor/sakura/pull/716) ([beru](https://github.com/beru))
+- \[拾い物です\] 背景画像表示の不透明度を設定出来るように変更 [\#704](https://github.com/sakura-editor/sakura/pull/704) ([ds14050](https://github.com/ds14050))


Issueのタイトルから生成していてこうなってしまっていると思いますが、[拾い物です] というのはユーザーからするとなんのこっちゃという感じなのでIssueのタイトルから削った方が良さそうですね。

PR #704 のタイトルから [拾い物です] というのを削りました。

CHANGELOG.mdを生成し直して修正しました。

beru · 2019-03-26T13:41:36Z

CHANGELOG.md

+### その他変更
+
+- githash.batでgitを探すようにする [\#794](https://github.com/sakura-editor/sakura/pull/794) ([berryzplus](https://github.com/berryzplus))
+- Feature/workaround for issue398 [\#792](https://github.com/sakura-editor/sakura/pull/792) ([berryzplus](https://github.com/berryzplus))


Feature/workaround for issue398 という説明だとページを辿らないと内容が分からないのでIssueのタイトルを変えた方が良いと思います。特定のファイルで描画が遅くなる問題への対策 とかで良いと思います。

PR #792 のタイトルを更新しました。

CHANGELOG.mdを生成し直して修正しました。

PR #792 のタイトルを更新しました。

Feature/workaround for issue398 の部分を日本語にしました。

CHANGELOG.mdを生成し直して修正しました。

takke · 2019-03-26T13:47:39Z

任意のタイミングで CHANGELOG.md を再生成したいのですが Appveyor の実行権限みたいなものもらえますか？ > @m-tmatma さん

m-tmatma · 2019-03-26T21:23:04Z

任意のタイミングで CHANGELOG.md を再生成したいのですが Appveyor の実行権限みたいなものもらえますか？ > @m-tmatma さん

サクラエディタの開発メンバーは全員権限がすでに付与されています。

https://ci.appveyor.com/login
にアクセスした後、"GitHub" のアイコンを押すと GitHub のアカウントでログインできるので
https://ci.appveyor.com/project/sakuraeditor/changelog-sakura
にアクセスして "New Build" を押せば (changelog-sakura の) master に対してビルドが走ります。

takke · 2019-03-27T00:31:27Z

https://ci.appveyor.com/login
にアクセスした後、"GitHub" のアイコンを押すと GitHub のアカウントでログインできるので
https://ci.appveyor.com/project/sakuraeditor/changelog-sakura
にアクセスして "New Build" を押せば (changelog-sakura の) master に対してビルドが走ります。

GitHub のアカウントでログインし直してみたところ、左上のドロップダウンに sakuraeditor が出てきたのでログイン自体は問題なさそうなのですが、"New Build" ボタンは出てきませんでした。

自分のプロジェクトのほうは "New Build" ボタンがちゃんと出るのですが・・・。

ひとまず早朝のビルド結果を使って作業進めますね。

takke · 2019-03-27T01:12:40Z

最新の CHANGELOG.md を反映しました。

1点補足ですが、#808 が当初の CHANGELOG.md に含まれていなかったので追加しました。

今朝自動生成されたCHANGELOG.md では、#808 が Unreleased に含まれていますが、手動で v2.4.0-alpha1 に移動しています。

これは、「v2.4.0-alpha1 タグを #808 のマージコミットに付けた」のですが、「#808 はマージコミット後に閉じられた」ため、時間的に v2.4.0-alpha1 タグ以降のものと判定されてしまったためです。

今後はこの現象を回避するために、よくあるOSSの手順と同様に、

バージョンの変更コミット (いわゆる bump up version number to XXX)
タグ打ち
CHANGELOG.md 生成と更新
リリース

という手順を辿るのが良さそうです。

あれ？でもサクラエディタの場合はリリース前のバージョン変更コミットいらないのかな？既に v2.4.0 になってますもんね。

KENCHjp · 2019-03-27T01:13:39Z

ここは、運用しながらですね。
以前の議論で開発中かどうかわかった方がいいよねっていうのと、バージョン上げないと今のリリースと被るからどうするかって議論がありまして。

KENCHjp · 2019-03-27T01:16:52Z

https://ci.appveyor.com/project/sakuraeditor/changelog-sakura
こちら私も、NEW BUILDでますね。
userとadminの違いのようですね。

KENCHjp · 2019-03-27T01:17:42Z

run build とview権限をuser（git hub側のdeveloper）に付与してみました。

takke · 2019-03-27T01:22:10Z

以前の議論で開発中かどうかわかった方がいいよねっていうのと、バージョン上げないと今のリリースと被るからどうするかって議論がありまして。

ですよね！いま v2.4.0 と表記されてるのは分かりやすくて素敵なんですが、リリース時にどうバージョン表記するかは未定って感じですかね。

リリース版は AppVeyor ではなくどなたかがローカルでビルドしたものを使うとして、そのときは AppVeyor 版とは違うバージョン表記になるので "v2.4.0-alpha1" みたいにバージョン情報に表示できるという理解でいいですかね(以前ソースざっくり読んだ限りそう書かれていたかと)。
そのためにソースコードを修正するとしたら、そのコミットをどこかでしていただきたい感じです。
今回はとりあえず AppVeyor から持ってきたものを公開でも全然かまわないです。

リリース担当者必要っすね。。

takke · 2019-03-27T01:24:44Z

run build とview権限をuser（git hub側のdeveloper）に付与してみました。

なるほど、ありがとうございます！
"NEW BUILD" ボタン出ました！

KENCHjp · 2019-03-27T01:28:38Z

リリース担当者必要っすね。。

wikiでリリース時にやること(案)のところ具体的な手順って起してもらうのお願いできますか？
（できたらサクラエディタ使ってやれるのがベスト）
手が空いてる時でいいです全然すぐでなくていいです。
私でもできるように(笑)

ゆくゆくはどうにかして自動化したいところですけど、あまり複雑にするのもあれなので。

KENCHjp · 2019-03-27T01:35:46Z

今回はとりあえず AppVeyor から持ってきたものを公開でも全然かまわないです。

本チャンも、 AppVeyorから持ってこようと思っておりますです。個人の環境の生成物だと差異があったときに拾えないかなと思うので。

ここのコメント伸ばさずに、
sakura-editor/management-forum#52
こっちに継続したほうがいい話かもしれませんが。

m-tmatma · 2019-03-27T13:12:26Z

今回はとりあえず AppVeyor から持ってきたものを公開でも全然かまわないです。

本チャンも、 AppVeyorから持ってこようと思っておりますです。個人の環境の生成物だと差異があったときに拾えないかなと思うので。

はい、ローカルビルドはあくまで開発用で、
公開用は appeyor ビルドを想定してます。

appeyor でビルドした時に定義される専用のマクロがあって
それを参照することによって、バージョン情報ダイアログで
どういうビルドかわかるようになっています。

githash.bat を見ていただくとわかると思います。

…om/project/sakuraeditor/changelog-sakura/builds/23483599

takke · 2019-03-31T07:29:09Z

CHANGELOG.md を生成し直して修正しました。次のバージョンは未定なので Unreleased としています。

v2.4.0-alpha1 は一応リリース済みなのでそろそろ CHANGELOG.md も master に取り込みたいです。。

m-tmatma

いいと思います。

m-tmatma · 2019-03-31T07:48:08Z

なお #818 をマージしたら md ファイルの変更を appveyor は無視してくれるようになるので
不必要にビルドが走らなくなります。

CHANGELOG.md を追加する

takke force-pushed the add_changelog branch from 9737228 to 57a72fa Compare March 25, 2019 04:00

takke added 2 commits March 25, 2019 13:03

add CHANGELOG.md from appveyor https://ci.appveyor.com/project/sakura…

3bfdc3e

…editor/changelog-sakura/builds/23319217

fix version, remove issues

ee7be35

takke force-pushed the add_changelog branch from 57a72fa to ee7be35 Compare March 25, 2019 04:03

berryzplus mentioned this pull request Mar 25, 2019

[WIP] HTML ヘルプの文字コードを UTF-8 BOM 付きに変換する #807

Closed

fix version as v2.4.0-alpha1

4e80cf6

takke force-pushed the add_changelog branch from 09ef3ea to 4e80cf6 Compare March 26, 2019 07:06

beru reviewed Mar 26, 2019

View reviewed changes

KENCHjp mentioned this pull request Mar 27, 2019

リリースに関して sakura-editor/management-forum#52

Closed

PR のタイトル修正を反映、sakura-editor#808 が含まれていなかったのを追加

d36e82b

KENCHjp added management 運営に関する話題【ChangeLog除外】 Release Release作業チケット【ChangeLog除外】 labels Mar 28, 2019

takke changed the title ~~Add CHANGELOG.md~~ CHANGELOG.md を追加する Mar 28, 2019

PR sakura-editor#792 の変更反映、sakura-editor#812 追加 https://ci.appveyor.c…

d9e2b9f

…om/project/sakuraeditor/changelog-sakura/builds/23483599

m-tmatma approved these changes Mar 31, 2019

View reviewed changes

takke merged commit 2f5395a into sakura-editor:master Mar 31, 2019

takke deleted the add_changelog branch March 31, 2019 13:05

m-tmatma added this to the v2.4.0 milestone Apr 28, 2019

HoppingTappy pushed a commit to HoppingTappy/sakura that referenced this pull request Jun 11, 2019

Merge pull request sakura-editor#813 from takke/add_changelog

ac7eb37

CHANGELOG.md を追加する

CHANGELOG.md を追加する #813

CHANGELOG.md を追加する #813

Conversation

takke commented Mar 25, 2019

m-tmatma commented Mar 25, 2019

takke commented Mar 25, 2019

KENCHjp commented Mar 25, 2019

KENCHjp commented Mar 25, 2019

m-tmatma commented Mar 25, 2019

takke commented Mar 26, 2019

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

beru Mar 26, 2019 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

takke commented Mar 26, 2019

m-tmatma commented Mar 26, 2019

takke commented Mar 27, 2019

takke commented Mar 27, 2019

KENCHjp commented Mar 27, 2019 • edited Loading

KENCHjp commented Mar 27, 2019

KENCHjp commented Mar 27, 2019 • edited Loading

takke commented Mar 27, 2019

takke commented Mar 27, 2019

KENCHjp commented Mar 27, 2019

KENCHjp commented Mar 27, 2019

m-tmatma commented Mar 27, 2019

takke commented Mar 31, 2019

m-tmatma left a comment

Choose a reason for hiding this comment

m-tmatma commented Mar 31, 2019

beru Mar 26, 2019 •

edited

Loading

KENCHjp commented Mar 27, 2019 •

edited

Loading

KENCHjp commented Mar 27, 2019 •

edited

Loading