-
Notifications
You must be signed in to change notification settings - Fork 7
feat: improve changelog formatting #127
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
Conversation
Signed-off-by: skjnldsv <[email protected]>
|
||
function groupPRsByRepo($prs) { | ||
$grouped = []; | ||
foreach ($prs as $pr) { |
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.
Should we sort $prs
by title here to have all commits on the same type together (fix, feat…)?
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.
Would be nice yeah!
And maybe remove chores and ci commits? :)
Forum / markdown
BEFORE
@nextcloud/vue
to v8.25.1 (server#52297)PHP_INT_MAX
where needed (server#52513)download
attribute when toggling checkbox (server#52516)has-preview
is false (server#52818)Node::copy()
(server#53158)view local
dialog was just closed (server#53176)AFTER
## Nextcloud 30.0.10server
activity
app_api
bruteforcesettings
circles
files_pdfviewer
logreader
nextcloud_announcements
notifications
photos
serverinfo
suspicious_login
text
twofactor_nextcloud_notification
twofactor_totp
updater
viewer