-
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 <skjnldsv@protonmail.com>
|
|
||
| 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/vueto v8.25.1 (server#52297)PHP_INT_MAXwhere needed (server#52513)downloadattribute when toggling checkbox (server#52516)has-previewis false (server#52818)Node::copy()(server#53158)view localdialog 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