[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#29137] [PATCH] gnu: Add python-clf.
From: |
Marius Bakke |
Subject: |
[bug#29137] [PATCH] gnu: Add python-clf. |
Date: |
Sun, 05 Nov 2017 17:16:45 +0100 |
User-agent: |
Notmuch/0.25.1 (https://notmuchmail.org) Emacs/25.3.1 (x86_64-pc-linux-gnu) |
Oleg Pykhalov <address@hidden> writes:
> * gnu/packages/web.scm (python-clf): New variable.
[...]
> +(define-public python-clf
> + (package
> + (name "python-clf")
> + (version "0.5.7")
> + (source
> + (origin
> + (method url-fetch)
> + (uri (pypi-uri "clf" version))
> + (sha256
> + (base32
> + "0zlkzqnpz7a4iavsq5vaz0nf5nr7qm5znpg1vlpz6rwnx6hikjdb"))))
> + (build-system python-build-system)
> + (propagated-inputs
> + `(("python-docopt" ,python-docopt)
> + ("python-pygments" ,python-pygments)
> + ("python-requests" ,python-requests)))
> + (home-page "https://github.com/ncrocfer/clf")
I noticed in the '.travis.yml' that it has a test suite that can be run
with 'nosetests'. Can you try overriding the check phase with one that
runs it? That way we are likely to catch regressions early.
> + (synopsis "Search code snippets on @url{commandlinefu.com}")
> + (description "@code{clf} is a command line tool for searching code
> +snippets on @url{commandlinefu.com}")
Note: descriptions should end with a punctuation. I think I'd also add
http:// to the @url{}s here for clarity.
The rest LGTM.
signature.asc
Description: PGP signature