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.

Sign up for GitHub

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

Jump to bottom

Beta3までのチェンジログ反映。 #1153

Merged
merged 3 commits into from
Feb 3, 2020
Merged

Beta3までのチェンジログ反映。 #1153

merged 3 commits into from
Feb 3, 2020

Conversation

KENCHjp
Copy link
Member

@KENCHjp KENCHjp commented Jan 14, 2020

PR の目的

Beta3 までの反映。

カテゴリ

  • ドキュメント修正
  • CI関連

PR の背景

リリース前にチェンジログを反映。

PR のメリット

PR のデメリット (トレードオフとかあれば)

PR の影響範囲

関連チケット

参考資料

@KENCHjp
Copy link
Member Author

KENCHjp commented Jan 14, 2020

すいません、またPR元のブランチ名をmasterのまま作業してました。

@k-takata
Copy link
Member

すいません、またPR元のブランチ名をmasterのまま作業してました。

まあ、作業している当人が混乱やミスをしなければ別にいいのですが。（周りの人には直接的な影響はないので。）

@KENCHjp KENCHjp added Release Release作業チケット【ChangeLog除外】 document ドキュメント and removed Release Release作業チケット【ChangeLog除外】 labels Jan 14, 2020
@beru beru added the no-changelog 【ChangeLog除外】 label Jan 16, 2020
beru
beru reviewed Jan 16, 2020
View reviewed changes
CHANGELOG.md Outdated Show resolved Hide resolved
beru
beru reviewed Jan 16, 2020
View reviewed changes
CHANGELOG.md Outdated Show resolved Hide resolved
beru
beru reviewed Jan 16, 2020
View reviewed changes
CHANGELOG.md Outdated Show resolved Hide resolved
beru
beru reviewed Jan 17, 2020
View reviewed changes
CHANGELOG.md Outdated Show resolved Hide resolved
beru
beru reviewed Jan 17, 2020
View reviewed changes
CHANGELOG.md Outdated Show resolved Hide resolved
beru
beru reviewed Jan 17, 2020
View reviewed changes
CHANGELOG.md Outdated Show resolved Hide resolved
beru
beru reviewed Jan 17, 2020
View reviewed changes
CHANGELOG.md Show resolved Hide resolved
beru
beru reviewed Jan 17, 2020
View reviewed changes
CHANGELOG.md Outdated
- \#780 でのCNativeW の初期状態での仕様を取り消して、 \#948 の不具合を修正する [\#949](https://github.com/sakura-editor/sakura/pull/949) ([m-tmatma](https://github.com/m-tmatma))
- DISABLED テストのサンプルを追加する [\#943](https://github.com/sakura-editor/sakura/pull/943) ([m-tmatma](https://github.com/m-tmatma))
- HTML の Help 中の JavaScript の文字コードを UTF-8 に修正 [\#939](https://github.com/sakura-editor/sakura/pull/939) ([m-tmatma](https://github.com/m-tmatma))
- Issue Template を改善する [\#933](https://github.com/sakura-editor/sakura/pull/933) ([m-tmatma](https://github.com/m-tmatma))
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

#933 は github 上にしか影響しない事なので changelog には載せない方が良いと思います。

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

つけました。

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Issue Template に関しては CI というラベルを付けるのは適切ではないと思うので外して代わりに management のラベルを付けておきました。まぁこれも適切では無いと言われたら何ともですが…。

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

環境ってラベルも考えたんですけど、no-changelogと一緒でなんでもかんでもそれになりそうなので止めました。。
おいおいですかねぇ。

beru
beru reviewed Jan 17, 2020
View reviewed changes
CHANGELOG.md
- DISABLED テストのサンプルを追加する [\#943](https://github.com/sakura-editor/sakura/pull/943) ([m-tmatma](https://github.com/m-tmatma))
- HTML の Help 中の JavaScript の文字コードを UTF-8 に修正 [\#939](https://github.com/sakura-editor/sakura/pull/939) ([m-tmatma](https://github.com/m-tmatma))
- Issue Template を改善する [\#933](https://github.com/sakura-editor/sakura/pull/933) ([m-tmatma](https://github.com/m-tmatma))
- .gitignore をソートする [\#927](https://github.com/sakura-editor/sakura/pull/927) ([m-tmatma](https://github.com/m-tmatma))
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

#927 はリリース物には関係無い事なので changelog には載せない方が良いと思います。

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

つけました。

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CI というラベルが付いていますが、.gitignore ファイルは git の設定ファイルの1つで CI 関連のファイルではないです。

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

環境って感じでCIにしちゃいましたが、それっぽいのがないので、Gitってラベルを新たに作ってみましたがいかがでしょ？

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

確認しました。Description が現時点では空になっていますが、【ChangeLog除外】 という認識で良いでしょうか？

Copy link
Member Author

@KENCHjp KENCHjp Jan 20, 2020 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ChangeLog除外の認識ですが、ChangeLogの生成側を修正する必要ありますね。

beru
beru reviewed Jan 17, 2020
View reviewed changes
CHANGELOG.md Show resolved Hide resolved
beru
beru reviewed Jan 17, 2020
View reviewed changes
CHANGELOG.md Outdated Show resolved Hide resolved
@beru
Copy link
Contributor

beru commented Jan 17, 2020

changelog に載せるべきかどうか考えてみると、なんか色々なPRが目に付いてしまいました。
あまり深く考えない方がいいんでしょうかね。。

@berryzplus
Copy link
Contributor

うん、たぶん。

カッコいいチェンジログが自動生成で出来上がる

理想だけどそれは無理だと思います。

PRの単位が変更内容じゃなくて修正内容だから。

変更履歴に欲しいのは変更の概要だと思います。具体的な修正を書いてもあんまし意味がないです。結局、どこがどう変わったのかすべてを知りたい人ってほとんどいない気がします。

更新履歴ってたぶん、アップデートするかどうか判断する材料として使われると思います。何ができるようになって、何が出来なくなったのか。「不具合をたくさん直しました！」と書いてある更新履歴はよく見ますが、直した不具合ぜんぶ載せてるのは見たことないです。

たくさん並べて見てはじめて分かることがあるような気がします。

最初からその観点を持っていた人もいるし、持ってなかった人もいる(きゃー)

知らないことは体験して知ればいいように思います。

だから細かいことは一旦放置でいいかな、と。

@beru
Copy link
Contributor

beru commented Jan 17, 2020 •
edited
Loading

デフォルトではPRは changelog に載せないで、changelog というラベルを付けたPRだけ changelog に載せる方がきれいになるかもしれないですね。でもこのアイデアを運用するには時すでにお寿司状態かも…、no-changelog というラベルが付いたPRだらけだし。

@KENCHjp
Copy link
Member Author

KENCHjp commented Jan 17, 2020

チェンジログはだれが見るかって観点で、結構プログラム修正する人も見るんじゃないかなと思ったりもして、
外部仕様に限らずある程度内部仕様の変更も載せてもいいのかなと思ったりもしてます。CIとかGitHub周りは積極的に外そうかなってのが今のモチベです。
チェンジログきれいにするためにあまり工数かけたくないなというのが正直なところですが(笑)

@KENCHjp
Copy link
Member Author

KENCHjp commented Jan 17, 2020 •
edited
Loading

内部仕様変更と、外部仕様変更（ @beru さん案ですね）とかラベルつくって外部仕様変更だけチェンジログにしてもよかったのかもしれませんけど、
当初チェンジログのためって感じでは動いてなかったですからね。（チェンジログを意識してラベルつけしようとするとまた手数が一つ増えるので、なるべく意識しないでラベルつけしたいかんじはあります）
これも走りながら不都合があったらまた考える一つかもしれません。

@KENCHjp
Copy link
Member Author

KENCHjp commented Jan 17, 2020 •
edited
Loading

あと、チェンジログに載せないラベルがいくつかあったと思いますが、
CIとかreleaseとかはチェンジログに拾わないようにするのが正解かもしれませんね。
いったんno-changelogつけましたが、おっしゃる通りno-changelogだらけになるのは本望ではないですし。

m-tmatma
m-tmatma reviewed Jan 17, 2020
View reviewed changes
CHANGELOG.md Outdated
@@ -83,9 +167,11 @@
- 背景塗りつぶしにPatBltを使う Part1 [\#765](https://github.com/sakura-editor/sakura/pull/765) ([berryzplus](https://github.com/berryzplus))
- WinMain のコメントを修正する [\#764](https://github.com/sakura-editor/sakura/pull/764) ([m-tmatma](https://github.com/m-tmatma))
- Copyright を 2019 にする [\#761](https://github.com/sakura-editor/sakura/pull/761) ([m-tmatma](https://github.com/m-tmatma))
- grep で 除外ファイル、除外フォルダが効かない のを修正 [\#758](https://github.com/sakura-editor/sakura/pull/758) ([m-tmatma](https://github.com/m-tmatma))
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ここはバグ修正からカテゴリが変わっていますが
何を変えましたか？

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ん、特になにもいじってないはず。

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

２回拾ってるみたいですね。

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

やりとりは多分チケットに残っているはず。

なんか、

リリースしてない新機能なんだから「バグ修正」ってしなくても良くね？

って話でタグ外してた気がします。

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

失礼、見間違えました、２回拾ってるのではなくて、バグ修正が無くなって、その他になってるんですね。
バグ修正軒並み消えてる感じかな。 @takke さん。

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

いま確認したところ確かに「バグ修正」のカテゴリが消えていますね。
github_changelog_generator のバージョンが変わったことが原因かもしれないですね。
取り急ぎ sakura-editor/changelog-sakura#28 を立てておきました。
あ、私に直せっていう話ですかね？ｗ

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

バグのラベルがいつの間にか(2019/8/24頃？) "bug" から "🐛bug🦋" に変わってたのでバグ修正扱いではなくなっていました。changelog-sakura のほうは sakura-editor/changelog-sakura#29 で修正したのでご確認ください。

Copy link
Member Author

@KENCHjp KENCHjp Jan 20, 2020 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

了解(笑)
お手数おかけしました。

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

修正はあれですが、原因特定はお願いしたかったところなんですけど、瞬殺だったみたいですね。　お声がけしてよかったっす。

@KENCHjp KENCHjp added question Further information is requested 【ChangeLog除外】 and removed question Further information is requested 【ChangeLog除外】 labels Jan 20, 2020
beru
beru previously approved these changes Jan 22, 2020
View reviewed changes
Copy link
Contributor

@beru beru left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OKだと思います。

changelogの自動生成には課題が色々あり一朝一夕で解決するものでは無いですね。

@KENCHjp
Copy link
Member Author

KENCHjp commented Jan 23, 2020

では、もういっかい取り直して再度Pushします。

@KENCHjp
Copy link
Member Author

KENCHjp commented Feb 3, 2020

再アップロードしたので、再レビューヨロです。

beru
beru approved these changes Feb 3, 2020
View reviewed changes
Copy link
Contributor

@beru beru left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OKだと思います。

@KENCHjp KENCHjp merged commit a2320cb into sakura-editor:master Feb 3, 2020
@m-tmatma m-tmatma added this to the v2.4.0 milestone Apr 19, 2020
HoppingTappy pushed a commit to HoppingTappy/sakura that referenced this pull request Jun 16, 2020
@KENCHjp
Merge pull request sakura-editor#1153 from KENCHjp/master
6efe33d 
ありがとうございます。
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@m-tmatma m-tmatma m-tmatma left review comments

@berryzplus berryzplus berryzplus left review comments

@takke takke takke left review comments

@beru beru beru approved these changes

Assignees
No one assigned
Labels
document ドキュメント no-changelog 【ChangeLog除外】
Projects
None yet
Milestone
v2.4.0
Development

Successfully merging this pull request may close these issues.
6 participants
@KENCHjp @k-takata @beru @berryzplus @takke @m-tmatma