-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Update CHANGELOG.md for 2.3.2 #4563
Conversation
CHANGELOG.md
Outdated
@@ -3,6 +3,14 @@ | |||
## [2.3.2](https://launchpad.net/mixxx/+milestone/2.3.2) (Unreleased) | |||
|
|||
* Improve robustness of file type detection by considering the actual MIME type of the content. [lp:1445885](https://bugs.launchpad.net/mixxx/+bug/1445885) [#4356](https://github.com/mixxxdj/mixxx/pull/4356) [#4357](https://github.com/mixxxdj/mixxx/pull/4357) | |||
* Fix/Improve Appstream metainfo |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should go into a packaging section because it affects package maintainers.
CHANGELOG.md
Outdated
* Playlist: enable sorting by color [#4352](https://github.com/mixxxdj/mixxx/pull/4352) [lp:1945976](https://bugs.launchpad.net/mixxx/+bug/1945976) | ||
* CMake: Adjust the download directory and name of external dependencies [#4511](https://github.com/mixxxdj/mixxx/pull/4511) | ||
* Fix clang compiler warnings [#4551](https://github.com/mixxxdj/mixxx/pull/4551) [#4553](https://github.com/mixxxdj/mixxx/pull/4553) | ||
* ClangFormat: Enable c++17 formatting [#4557](https://github.com/mixxxdj/mixxx/pull/4557) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think that does not belong into the changelog because it's irrelevant to the user.
CHANGELOG.md
Outdated
[#4349](https://github.com/mixxxdj/mixxx/pull/4349) | ||
* Playlist: enable sorting by color [#4352](https://github.com/mixxxdj/mixxx/pull/4352) [lp:1945976](https://bugs.launchpad.net/mixxx/+bug/1945976) | ||
* CMake: Adjust the download directory and name of external dependencies [#4511](https://github.com/mixxxdj/mixxx/pull/4511) | ||
* Fix clang compiler warnings [#4551](https://github.com/mixxxdj/mixxx/pull/4551) [#4553](https://github.com/mixxxdj/mixxx/pull/4553) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think this needs to be mentioned in the changelog.
CHANGELOG.md
Outdated
[#4344](https://github.com/mixxxdj/mixxx/pull/4344) | ||
[#4346](https://github.com/mixxxdj/mixxx/pull/4346) | ||
[#4349](https://github.com/mixxxdj/mixxx/pull/4349) | ||
* Playlist: enable sorting by color [#4352](https://github.com/mixxxdj/mixxx/pull/4352) [lp:1945976](https://bugs.launchpad.net/mixxx/+bug/1945976) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
* Playlist: enable sorting by color [#4352](https://github.com/mixxxdj/mixxx/pull/4352) [lp:1945976](https://bugs.launchpad.net/mixxx/+bug/1945976) | |
* Playlist: Enable sorting by color [#4352](https://github.com/mixxxdj/mixxx/pull/4352) [lp:1945976](https://bugs.launchpad.net/mixxx/+bug/1945976) |
CHANGELOG.md
Outdated
[#4346](https://github.com/mixxxdj/mixxx/pull/4346) | ||
[#4349](https://github.com/mixxxdj/mixxx/pull/4349) | ||
* Playlist: enable sorting by color [#4352](https://github.com/mixxxdj/mixxx/pull/4352) [lp:1945976](https://bugs.launchpad.net/mixxx/+bug/1945976) | ||
* CMake: Adjust the download directory and name of external dependencies [#4511](https://github.com/mixxxdj/mixxx/pull/4511) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@uklotzde is this relevant to package maintainers where build servers mustn't access the internet?
If so, this should go into a packaging section. If not, this doesn't need to be mentioned at all.
Done |
Why are we adding PRs to the 2.3.1 and older changelog sections retroactively? |
I have added #4070, because it was int my Todo list for 2.4 and it turns out it was already part of 2.3.1 That's the only reason. I actually don't mind to keep or remove it. |
Thanks for the explanation. Going back and adding PRs we forgot to put into the changelog is fine IMO. |
This updates the CHANGELOG for a 2.3.2 release.