[pve-devel] [PATCH manager] fix #5787: ui: display vm description in confirm dialogs
Fiona Ebner
f.ebner at proxmox.com
Tue Oct 22 12:39:22 CEST 2024
Am 22.10.24 um 08:58 schrieb Thomas Lamprecht:
> Am 21/10/2024 um 14:17 schrieb Timothy Nicholson:
>> Signed-off-by: Timothy Nicholson <t.nicholson at proxmox.com>
>> ---
>>
>> This patch adds a new function to the PVE Utils that formats a task confirmation message to display in confirm dialogs, for example when shutting down a VM. As requested by the Bugzilla entry #5787, the message now includes the VM name in addition to the VM ID.
>
> why isn't above stated in the commit message but only as patch note?
>
Also please note (from [0]):
> Make sure the line-length (text-width, column count) of the commit's message is not longer than 72 characters.
which is a good limit for readability in mails in general :)
[0]:
https://pve.proxmox.com/wiki/Developer_Documentation#Commits_and_Commit_Messages
More information about the pve-devel
mailing list