guix-patches
[Top][All Lists]
Advanced

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

[bug#53765] [PATCH 12/17] gnu: Add clojure-com-cognitect-http-client.


From: Maxime Devos
Subject: [bug#53765] [PATCH 12/17] gnu: Add clojure-com-cognitect-http-client.
Date: Mon, 07 Feb 2022 21:29:38 +0100
User-agent: Evolution 3.38.3-1

Reily Siegel schreef op do 03-02-2022 om 19:25 [-0500]:
> +    (synopsis "HTTP Client for Clojure")

Why is ‘Client’ capitalised here?

> +    (description "HTTP client for Clojure.")

This seems rather misleading, it is not its own HTTP client
but rather a tiny wrapper around jetty's HTTP client that
makes the API a bit less Java-y and more Clojure-y.

I would mention something along the lines

(description "This package is a Clojure wrapper around Jetty's HTTP
client for Java.  While Jetty is a Java package and hence its API
follows Java conventions, this Clojure wrapper follows Clojure API
conventions.")

Greetings,
Maxime.

Attachment: signature.asc
Description: This is a digitally signed message part


reply via email to

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