guix-devel
[Top][All Lists]
Advanced

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

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


From: Ricardo Wurmus
Subject: Re: [PATCH 01/10] gnu: Add python-pytest-cov.
Date: Wed, 30 Dec 2015 21:45:42 +0100
User-agent: mu4e 0.9.13; emacs 24.5.1

Efraim Flashner <address@hidden> writes:

> On Wed, 30 Dec 2015 15:10:06 +0100
> Ricardo Wurmus <address@hidden> wrote:
>
>> Efraim Flashner <address@hidden> writes:
>> 
>>  [...]  
>>  [...]  
>> 
>> 
>>  [...]  
>> 
>>  [...]  
>> 
>> > diff --git a/gnu/packages/python.scm b/gnu/packages/python.scm
>> > index 6af2bce..50137bd 100644
>> > --- a/gnu/packages/python.scm
>> > +++ b/gnu/packages/python.scm
>> > @@ -1632,18 +1632,18 @@ and many external plugins.")
>> >          (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.")
>> > +    (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 @code{load} and @code{each} modes.  It also
>> > +supports coverage of subprocesses.")
>> >    (license license:expat)))  
>> 
>> Why has the indentation changed for all lines since ‘(build-system
>> ...)’?  The description is clearer now, thanks.
>> 
>> ~~ Ricardo
>
> before it was
> (define-public python-pytest-cov
>   (package
>     (name "python-pytest-cov")
> ...
>   (build-system python-build-system)
> ...
>
> So I had it lined up with package instead of indented 2 more spaces.

Ah, I see, and someone already pushed the wrong version.  The commit
summary should probably be updated to mention this indentation change.

~~ Ricardo



reply via email to

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