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

CI: Changelogの編集対象をマージ前に確認する #12772

Closed
anatawa12 opened this issue Dec 24, 2023 · 7 comments · Fixed by #12963
Closed

CI: Changelogの編集対象をマージ前に確認する #12772

anatawa12 opened this issue Dec 24, 2023 · 7 comments · Fixed by #12963
Assignees
@samunohito
Labels
♻CI/CD 🛠️Dev Development of Misskey itself

Comments

@anatawa12
Copy link
Member

anatawa12 commented Dec 24, 2023 •
edited
Loading

リリースをまたいだプルリクをマージするときに古いリリース部分にchangelogを追加してしまってるのを気づかずにリリースしちゃってる事例がまぁまぁ(メジャーのたびに一回くらいづつ)あるので何かしらチェックを入れられると良さそうと思いました。
@syuilo syuilo added the 🛠️Dev Development of Misskey itself label Dec 24, 2023
@samunohito
Copy link
Member

ディレクトリ切って、そのなかにリリースごとのファイルを作るとか…？

@syuilo
Copy link
Member

syuilo commented Dec 24, 2023

YOSASOU

@syuilo
Copy link
Member

syuilo commented Dec 24, 2023

あー待てよ

@anatawa12
Copy link
Member Author

普通にCIでdiff見るでもいい気はする。
リリース時に結合とかバージョン名にリネームという手もあるけど

@syuilo
Copy link
Member

syuilo commented Dec 24, 2023 •
edited
Loading

ディレクトリ切って、そのなかにリリースごとのファイルを作るとか…？

それやったとしても

リリースをまたいだプルリクをマージするときに古いリリース部分にchangelogを追加してしま

は解決できないわね

@samunohito
Copy link
Member

想定とは違うファイルに変更が入るので目が届きやすくなるかな？と思ったりしてました
根本的にやるなら、もうCIパワーでどうにかするしかなさそうですね…

@samunohito samunohito moved this to Todo in [CI] TODO Dec 24, 2023
@tamaina
Copy link
Contributor

tamaina commented Dec 26, 2023

Related to #12807 ?

@samunohito samunohito moved this from Todo to In Progress in [CI] TODO Jan 10, 2024
@samunohito samunohito self-assigned this Jan 10, 2024
@samunohito samunohito mentioned this issue Jan 10, 2024
Merged
5 tasks
@github-project-automation github-project-automation bot moved this from In Progress to Done in [CI] TODO Jan 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@samunohito samunohito

Labels
♻CI/CD 🛠️Dev Development of Misskey itself
Projects
No open projects
[CI] TODO
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat(CI): CHANGELOG.mdの追記個所をチェックするCIを追加 samunohito/misskey
4 participants
@syuilo @tamaina @anatawa12 @samunohito