It will be great if pursuivant only displays the most important information about commit, e.g. the author, the subject line and the link. Omitting the commit message will make pursuivant less noisy on the channel.
Description
Description
Comment Actions
I'm afraid the tooling that supports Pursuivant at this time only allows for the same message format to be used across all channels.
We could certainly consider reducing the number of commit message lines printed on IRC though (we allow a maximum of 5 now).
See https://invent.kde.org/sysadmin/irc-notifications/-/blob/master/gateway/cia-smtp.py#L36 for the involved code.
Comment Actions
Bhusan said that he will take a look at this later. But if it's okay, we can change the message format so it's a single line with the most important information, e.g. <repo> <branch> <author> <subject line> <link>(<modified files>). Such a message format should produce compact messages and make pursuivant less noisy. :)
Comment Actions
In other words, someone just needs to re-shuffle the items in https://invent.kde.org/sysadmin/irc-notifications/-/blob/master/gateway/cia-smtp.py#L76. I can make a MR but I would need help with testing.
Comment Actions
Please reopen this if you have some ideas about how to proceed here.
Re-opening as I have a proposal how to proceed https://phabricator.kde.org/T14458#256006
Comment Actions
Following discussion this morning, we've concluded that while we would be happy to change this, because it has global impact on all KDE projects this should be discussed first within the broader community.
Please open a thread to discuss this on kde-community/kde-devel. Once the thread has concluded we can look at sorting this out.
Comment Actions
@ngraham started a relevant discussion recently in the plasma-devel mailing list https://mail.kde.org/pipermail/plasma-devel/2021-May/120182.html. So, I guess we can continue this discussion there.
Comment Actions
plasma-devel is not a global community forum, so that thread is not suitable for discussing changing the format of those messages I'm afraid.
It needs to be a generic list such as kde-devel.
Comment Actions
I started a new thread in kde-devel (https://mail.kde.org/pipermail/kde-devel/2021-May/000472.html) and there have been no objections. I also received two supportive comments in the Kirigami chatroom yesterday.
Comment Actions
Thanks for the update - let's give this a few days so people have a chance to see this before we change anything.
Comment Actions
The change to implement this has been staged in https://invent.kde.org/sysadmin/irc-notifications/-/merge_requests/3
Comment Actions
This has now been deployed.
Projects that wish to use shorter lines may now set "short_message=True" in their IRC Channel Commit Gateway configuration.
Comment Actions
You can change it for any channel by committing the appropriate change to the config in the sysadmin/irc-notifications repository.
Please ensure it is discussed with the members of the channel first.
Comment Actions
Sent a merge request with @mart CC'd: https://invent.kde.org/sysadmin/irc-notifications/-/merge_requests/4