unofficial mirror of bug-guix@gnu.org 
 help / color / mirror / code / Atom feed
* bug#61013: guix style indentation problem in (list ...)
@ 2023-01-23  0:38 Maxim Cournoyer
  2023-03-07 13:59 ` Ludovic Courtès
  0 siblings, 1 reply; 3+ messages in thread
From: Maxim Cournoyer @ 2023-01-23  0:38 UTC (permalink / raw)
  To: 61013

Hello,

I recently stumbled on this indentation produced with guix style:

--8<---------------cut here---------------start------------->8---
    (arguments
     (list
           ;; Disable tests that use the network and non-free assets.
           #:test-flags #~(list "-m" "not external and not samples")))
--8<---------------cut here---------------end--------------->8---

While the conventional way would be to indent it as:

--8<---------------cut here---------------start------------->8---
    (arguments
     (list
      ;; Disable tests that use the network and non-free assets.
      #:test-flags #~(list "-m" "not external and not samples")))
--8<---------------cut here---------------end--------------->8---

-- 
Thanks,
Maxim




^ permalink raw reply	[flat|nested] 3+ messages in thread

* bug#61013: guix style indentation problem in (list ...)
  2023-01-23  0:38 bug#61013: guix style indentation problem in (list ...) Maxim Cournoyer
@ 2023-03-07 13:59 ` Ludovic Courtès
  2023-03-07 18:31   ` Maxim Cournoyer
  0 siblings, 1 reply; 3+ messages in thread
From: Ludovic Courtès @ 2023-03-07 13:59 UTC (permalink / raw)
  To: Maxim Cournoyer; +Cc: 61013-done

Hello,

Maxim Cournoyer <maxim.cournoyer@gmail.com> skribis:

> I recently stumbled on this indentation produced with guix style:
>
>     (arguments
>      (list
>            ;; Disable tests that use the network and non-free assets.
>            #:test-flags #~(list "-m" "not external and not samples")))
>
>
> While the conventional way would be to indent it as:
>
>     (arguments
>      (list
>       ;; Disable tests that use the network and non-free assets.
>       #:test-flags #~(list "-m" "not external and not samples")))

Fixed in 0a37921d851b94aef6900214098dc5bd62b46e26, thanks!

Ludo’.




^ permalink raw reply	[flat|nested] 3+ messages in thread

* bug#61013: guix style indentation problem in (list ...)
  2023-03-07 13:59 ` Ludovic Courtès
@ 2023-03-07 18:31   ` Maxim Cournoyer
  0 siblings, 0 replies; 3+ messages in thread
From: Maxim Cournoyer @ 2023-03-07 18:31 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: 61013-done

Hi Ludo,

Ludovic Courtès <ludo@gnu.org> writes:

> Hello,
>
> Maxim Cournoyer <maxim.cournoyer@gmail.com> skribis:
>
>> I recently stumbled on this indentation produced with guix style:
>>
>>     (arguments
>>      (list
>>            ;; Disable tests that use the network and non-free assets.
>>            #:test-flags #~(list "-m" "not external and not samples")))
>>
>>
>> While the conventional way would be to indent it as:
>>
>>     (arguments
>>      (list
>>       ;; Disable tests that use the network and non-free assets.
>>       #:test-flags #~(list "-m" "not external and not samples")))
>
> Fixed in 0a37921d851b94aef6900214098dc5bd62b46e26, thanks!

Awesome, that was quick!  I'll try to use it more regularly and reports
of any problematic behavior.

-- 
Thanks,
Maxim




^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2023-03-07 18:32 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-01-23  0:38 bug#61013: guix style indentation problem in (list ...) Maxim Cournoyer
2023-03-07 13:59 ` Ludovic Courtès
2023-03-07 18:31   ` Maxim Cournoyer

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).