guix-patches
[Top][All Lists]
Advanced

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

[bug#28583] [PATCH 2/7] gnu: Add ruby-multi-json.


From: Christopher Baines
Subject: [bug#28583] [PATCH 2/7] gnu: Add ruby-multi-json.
Date: Sat, 30 Sep 2017 09:21:44 +0100

On Wed, 27 Sep 2017 22:03:32 +1000
Ben Woodcroft <address@hidden> wrote:

> On 25/09/17 03:17, Christopher Baines wrote:
> > * gnu/packages/ruby.scm (ruby-multi-json): New variable.  
> [..]
> > +    (arguments
> > +     '(#:tests? #f)) ;; No testsuite.  
> No testsuite included in distributed gem, rather.

I've updated the comment.

> > +    (synopsis
> > +     "Common interface to multiple JSON libraries for Ruby")
> > +    (description
> > +     "This package provides a common interface to multiple JSON
> > libraries, +including Oj, Yajl, the JSON gem (with C-extensions),
> > the pure-Ruby JSON gem, +NSJSONSerialization, gson.rb, JrJackson,
> > and OkJson.")
> > +    (home-page
> > +     "http://github.com/intridea/multi_json";)  
> Both home-page and synopsis could be moved to the line before.

And also updated the formatting.

> Otherwise LGTM.
> ben

Attachment: pgppwMVxm9q8S.pgp
Description: OpenPGP digital signature


reply via email to

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