RFR: 1660: Add confname field to slack log config [v2]
Erik Joelsson
erikj at openjdk.org
Tue Nov 15 22:06:44 UTC 2022
On Tue, 8 Nov 2022 09:00:47 GMT, Magnus Ihse Bursie <ihse at openjdk.org> wrote:
> The patch looks good. but I'm thinking if you should use some delimiter for the prefix, like `[` ... `]`. Or is the plan to make formatting for the prefix to be clearly marked as a prefix in the configuration? (I'm just worried that if this is not done properly it can be hard to distinguish the prefix from the message proper, making it look odd)
My plan was to have the prefix added completely raw and put any formatting in the config. That makes it more flexible and easier to change as well.
-------------
PR: https://git.openjdk.org/skara/pull/1413
More information about the skara-dev
mailing list