[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#34514] [PATCH 03/34] gnu: Add ruby-terraform.
From: |
Christopher Baines |
Subject: |
[bug#34514] [PATCH 03/34] gnu: Add ruby-terraform. |
Date: |
Tue, 19 Feb 2019 22:43:55 +0000 |
User-agent: |
mu4e 1.0; emacs 26.1 |
Ricardo Wurmus <address@hidden> writes:
> Christopher Baines <address@hidden> writes:
>
>> * gnu/packages/ruby.scm (ruby-terraform): New variable.
> […]
>> +(define-public ruby-terraform
>> + (package
>> + (name "ruby-terraform")
>> + (version "0.20.0")
>> + (source
>> + (origin
>> + (method url-fetch)
>> + (uri (rubygems-uri "ruby-terraform" version))
>> + (sha256
>> + (base32
>> + "1czpak24v570ysnk8waj1sblfm1vqh1sdwh934x4ai90ribhlbzf"))))
>
> The indentation isn’t right here.
Good spot, I've fixed this now.
>> + (build-system ruby-build-system)
>> + (arguments
>> + '(;; No included tests
>> + #:tests? #f))
>
> Could you please put this all on one line?
Ok. I've taken to putting the comment inline, as this is syntactically
next to the #:tests? argument and this makes it easier to add additional
arguments, as you don't have to shuffle the comment around. I'm happy to
just put it inline though.
signature.asc
Description: PGP signature
[bug#34514] [PATCH 07/34] gnu: Add ruby-multi-xml., Christopher Baines, 2019/02/17
[bug#34514] [PATCH 04/34] gnu: Add ruby-que., Christopher Baines, 2019/02/17
[bug#34514] [PATCH 11/34] gnu: Add ruby-warden-oauth2., Christopher Baines, 2019/02/17
[bug#34514] [PATCH 06/34] gnu: Add ruby-jwt., Christopher Baines, 2019/02/17
[bug#34514] [PATCH 08/34] gnu: Add ruby-oauth2., Christopher Baines, 2019/02/17