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

CHANGELOG.md の生成方法や取得方法を簡単に参照できるように markdown に説明を追加する #696

Merged
merged 5 commits into from
Dec 13, 2018

Conversation

m-tmatma
Copy link
Member

CHANGELOG.md の生成方法や取得方法を簡単に参照できるように markdown に説明を追加する

関連

@m-tmatma m-tmatma added this to the next release milestone Dec 11, 2018
@m-tmatma m-tmatma added the document ドキュメント label Dec 11, 2018
@takke
Copy link
Member

takke commented Dec 11, 2018

入ったばかりでこのプロジェクトの流儀が分かっていないので単なる質問なんですが、4コミットいずれも README.md に対する修正なので 1 コミットに rebase したほうがいいのかな、と思いました。

README.md Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
@berryzplus
Copy link
Contributor

しょうもないことで禿同…

タスクトレイのメニューをテストする方法でも良いと言ってみたりして 😄

@berryzplus
Copy link
Contributor

元コメに関しては「の記載」が要らない気がします。

URL の方法で確認できます。

で通じるような。

@m-tmatma
Copy link
Member Author

入ったばかりでこのプロジェクトの流儀が分かっていないので単なる質問なんですが、4コミットいずれも README.md に対する修正なので 1 コミットに rebase したほうがいいのかな、と思いました。

そこらへんは特に決まってないです。
1 commit にくっつけました。

@m-tmatma
Copy link
Member Author

元コメに関しては「の記載」が要らない気がします。

「の記載」を削除しました。

README.md Outdated

[生成した CHANGELOG.md は ここからダウンロードできます](https://ci.appveyor.com/project/sakuraeditor/changelog-sakura/branch/master/artifacts)

ダウンロードした `CHANGELOG.md` は [changelog-sakura の README.md](https://github.com/sakura-editor/changelog-sakura) の方法でローカルで確認できます。
Copy link
Contributor

Choose a reason for hiding this comment

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

リンク先ページ内の ダウンロードした CHANGELOG.md の確認方法 までリンクを張った方が分かりやすいかなと思いました。

Copy link
Member Author

Choose a reason for hiding this comment

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

そこは
sakura-editor/changelog-sakura#9
があるので、微妙です。

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

Choose a reason for hiding this comment

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

逆に確認方法をこっちに書いて、changelog-sakura 側はこっちへのリンク(かいっそ書かない)のほうが良さそうですね。こっちを先に見る人が圧倒的に多いと思うので。

Copy link
Member Author

Choose a reason for hiding this comment

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

単にtips なので、wiki に移動したらいいのかも

Copy link
Member Author

Choose a reason for hiding this comment

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

wiki へのリンクに変えました。

@takke
Copy link
Member

takke commented Dec 13, 2018

CHANGELOG.mdについて の wiki にラベル付けルールなどをまとめたので README.md からもリンクを貼っておいてもらいたいです。

@m-tmatma
Copy link
Member Author

CHANGELOG.mdについて の wiki にラベル付けルールなどをまとめたので README.md からもリンクを貼っておいてもらいたいです。

修正しました。

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.

問題無いと思います。

@m-tmatma m-tmatma merged commit d3dbb34 into sakura-editor:master Dec 13, 2018
@m-tmatma m-tmatma deleted the feature/markdown-changelog branch December 13, 2018 20:52
HoppingTappy pushed a commit to HoppingTappy/sakura that referenced this pull request Jun 11, 2019
…hangelog

CHANGELOG.md の生成方法や取得方法を簡単に参照できるように markdown に説明を追加する
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
document ドキュメント
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants