Opened 7 years ago
Closed 5 years ago
#2918 closed enhancement (wontfix)
Improve processes related to uploading a patch
Reported by: | flixos90 | Owned by: | |
---|---|---|---|
Milestone: | Priority: | normal | |
Component: | Trac | Keywords: | |
Cc: |
Description
As discussed in recent multisite office hours and then dev chat, it would be great if we could improve the processes that need to happen after uploading a patch to a Trac ticket.
The initial idea was to automatically add the has-patch
keyword once a new patch is uploaded and the keyword has not been set before. Even though a patch does not always fix the issue described in the ticket properly, the keyword should always be applied initially (before someone else possibly reviews the patch and changes the keywords again).
However, there was criticism about that idea expressed by @jorbin, since people sometimes upload a wrong patch, then delete it and reupload a new one. This is a valid point, however I'm not sure if that's even a problem. If a patch is deleted and then reuploaded with the same name, it should not matter. It's something to further think about though.
The alternative (or even complementary) approach is to somewhere show a note to briefly highlight what a contributor should do after uploading a patch, which currently is often not clear. It could be a notice such as "Please comment on the ticket, and briefly describe what the patch does. If you enhanced a previous patch, describe what you changed from that patch.". This is not only important for communication about the changes itself, but also so that people are pinged (as the patch upload itself does not trigger a ping).
Trac currently doesn't send email notifications after a patch has been uploaded, so if the keyword
has-patch
is added automatically then anyone who is watching the ticket will not receive a notification that a patch has been added to the ticket.As such going for the latter approach mentioned above would be required now to trigger a Trac notification email
A Trac update is available in #1606 that includes an update to send email notifications when an attachment is uploaded
Related: