Re: [PATCH 01/10] gnu: Add python-pytest-cov.

[Ricardo Wurmus]

Efraim Flashner <address@hidden> writes:

> From 645f979af579abb97a69b1a8814034f4c2d9b499 Mon Sep 17 00:00:00 2001
> From: Efraim Flashner <address@hidden>
> Date: Mon, 23 Nov 2015 17:12:52 +0200
> Subject: [PATCH 01/10] gnu: Add python-pytest-cov.

> * gnu/packages/python.scm (python-pytest-cov, python2-pytest-cov): New
> variables.
> ---
>  gnu/packages/python.scm | 28 ++++++++++++++++++++++++++++
>  1 file changed, 28 insertions(+)

> diff --git a/gnu/packages/python.scm b/gnu/packages/python.scm
> index d01c1d3..280f0c9 100644
> --- a/gnu/packages/python.scm
> +++ b/gnu/packages/python.scm
> @@ -1634,6 +1634,34 @@ and many external plugins.")
>  (define-public python2-pytest
>    (package-with-python2 python-pytest))
 
> +(define-public python-pytest-cov
> +  (package
> +    (name "python-pytest-cov")
> +    (version "2.2.0")
> +    (source
> +      (origin
> +        (method url-fetch)
> +        (uri (pypi-uri "pytest-cov" version))
> +        (sha256
> +          (base32
> +           "1lf9jsmhqk5nc4w3kzwglmdzjvmi7ajvrsnwv826j3bn0wzx8c92"))))
> +  (build-system python-build-system)
> +  (propagated-inputs
> +   `(("python-coverage" ,python-coverage)
> +     ("python-pytest" ,python-pytest)))
> +  (native-inputs
> +   `(("python-setuptools" ,python-setuptools)))
> +  (home-page "https://github.com/pytest-dev/pytest-cov";)
> +  (synopsis "Pytest plugin for measuring coverage")
> +  (description
> +   "Pytest-cov produces coverage reports.  It supports centralised testing 
> and
> +distributed testing in both load and each modes.  It also supports coverage
> +of subprocesses.")

I just noticed that I don’t understand the second sentence.  What does
“in both load and each modes” mean?

Also, I think we need to use American English, so even though
“centralised” looks perfectly fine to me, I believe it should be
“centralized”.

It would be great if you could clarify the description.  If you don’t
have push access, I can take care of pushing the patch set then.

~~ Ricardo