guix-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[bug#33099] gnu: Add r-aspi.


From: Laura Lazzati
Subject: [bug#33099] gnu: Add r-aspi.
Date: Tue, 23 Oct 2018 11:47:53 -0300

On Tue, Oct 23, 2018 at 9:53 AM Björn Höfling
<address@hidden> wrote:
>
> On Tue, 23 Oct 2018 07:16:00 -0300
> Laura Lazzati <address@hidden> wrote:
>
> >
>
> Hi Laura,
>
> I noted three things, sorry this wasn't mentioned before:
>
> 1) Your name in the commit is:
>
> Author: laura <address@hidden>  2018-10-23 06:59:22
>
> Is that intended or do you prefer the full name in capitals (i.e.
> "Laura Lazzati", as you have set in your emails)?
>
> If not, set the name correctly with
>
> git config --global user.name "Laura Lazzati"
It was intentional, but I have already corrected it.
>
> 2) Add a Copyright line for yourself, just copy any other from the file:
>
> ;;; Copyright © 2018 Name <email> ...
>
> 3) Change the Homepage to use HTTPS.
>
> Otherwise, LGTM ("Looks good to me").
>
> If you decide on 1) then I think a committer could add/correct these for
> you instead of endlessly sending out new patches.
>
> Thanks!
>
> Björn
Here goes hope the last version. When approved I have some questions
to ask :), most of them to improve future patches
Regards,
Laura

Attachment: 0001-gnu-Add-r-aspi.patch
Description: Text Data


reply via email to

[Prev in Thread] Current Thread [Next in Thread]