Re: [PATCH] Syntax: Added syntax for Python decorators.

[matteo_luigi_raso]

Oops, meant to send a reply to the list but accidentally sent it to Benno. I'll post the entire exchange.

>On Thursday, January 19th, 2023 at 7:01 AM, Benno Schulenberg <bensberg@telfort.nl>

>wrote:

>> This commit message is too verbose -- too much banter. Instead it would

>> be good to include a URL to a webpage that documents these "decorators".

>> For example: https://peps.python.org/pep-0318/. But maybe you have a

>> better one?

> Actually, it's probably not necessary to explain the syntax of Python

> decorators at all. I only put that wall of text in case you didn't know the

> syntax of the decorators, but since you do, there's no need to explain it.

>> color sea "@[[:alpha:]][[:alnum:]_.]*"

>>

>> I've chosen 'sea' as color, because for me 'sky' has too low a contrast

>> with the lightblue of function names. Does this color and regex work for

>> you?

>For the most part. Not a huge fan of the colour change, but it's easy

>enough to customize, so I'm not worried about it.

In response, Benno wrote:

>Please send your email to the list, not to me.

>Op 20-01-2023 om 08:59 schreef matteo_luigi_raso:

>> Actually, it's probably not necessary to explain the syntax of Python

>> decorators at all. I only put that wall of text in case you didn't know the

>> syntax of the decorators, but since you do, there's no need to explain it.

>I knew nothing about decorators -- I had to google for it. But it's not about me, it is about

>anyone that reads the git history. So the commit message /does/ need to explain concisely

>what decorators are, or at least needs to refer to its canonical documentation.

>Benno

Ok, I've added a link in the commit message to the official documentation for Python decorators.

Sorry about that mixup, I don't have much experience with mailing lists.

Sent with Proton Mail secure email.

0001-SYNTAX-Added-colouring-for-Python-decorators.patch
Description: Text Data