Subject: Re: [bug#74344] [PATCH v3 5/7] gnu: Add qtpromise.

From: Maxim Cournoyer <[email-masked]>
Date: Sun, 16 Feb 2025 15:44:47 +0900

Hi,

Leo Famulari <[email-masked]> writes:

> On Tue, Feb 11, 2025 at 07:50:49PM +0530, Sughosha via Guix-patches via wrote:
>> * gnu/packages/qt.scm (qtpromise): New variable.
>
>> +               (replace 'install
>
> Please add a brief code comment explaining why the install phase is
> replaced.
>
>> +    (description
>> +     "This package provides Promises/A+ implementation for Qt/C++.")
>
> How about this: "This package provides an Qt/C++ implementation of the
> Promises/A+ standard for the 'promises' programming paradigm."

Yes, and if available I'd turn Promises/A+ into a @url pointing to where
the standard is defined, so that users can easily learn more if they'd
like to.

-- 
Thanks,
Maxim