If you do git show e9f4a0a595eaa82571e1dea55d672ec82da5d2c8 --stat, you get
commit e9f4a0a595eaa82571e1dea55d672ec82da5d2c8
Author: Mateusz Paprocki <matt...@gmail.com>
Date: Wed May 4 18:26:49 2011 -0700
Fixed tests after keep_sign was removed
sympy/core/tests/test_evalf.py | 1 +
sympy/core/tests/test_expr.py | 12 +++---
sympy/core/tests/test_numbers.py | 14 +++++---
.../special/tests/test_gamma_functions.py | 11 +++---
.../special/tests/test_spec_polynomials.py | 3 +-
.../special/tests/test_spherical_harmonics.py | 5 ++-
sympy/integrals/tests/test_rationaltools.py | 14 ++++----
sympy/polys/tests/test_partfrac.py | 5 ++-
sympy/polys/tests/test_polyfuncs.py | 3 +-
sympy/polys/tests/test_polyroots.py | 8 ++--
sympy/series/tests/test_nseries.py | 6 ++--
sympy/series/tests/test_series.py | 4 +-
sympy/simplify/tests/test_simplify.py | 11 +++---
sympy/solvers/tests/test_ode.py | 34 ++++++++++----------
sympy/solvers/tests/test_polysys.py | 4 +-
sympy/solvers/tests/test_solvers.py | 7 ++--
16 files changed, 76 insertions(+), 66 deletions(-)
which tells you a little more. I think for this commit, what is given
is fine. Maybe it could have referenced an issue number, but the
commit is just changing little things from the keep_sign behavior.
There's no need to meticulously document all of them. If you want to
know more, just look at the diff already.
And by the way, little changes like this don't need to go in the
release notes. In fact, keep_sign was added and removed entirely
since the last release, so I don't think it even needs to be mentioned
at all.
Aaron Meurer
On Thu, May 12, 2011 at 10:33 PM, Saptarshi Mandal
<sapta.iit...@gmail.com> wrote:
>> You can use `git log --name-status` to have git show you the files
>> that were modified.
>
> I'll take the previous example, so in one of the commits if I do a git
> log --name-status
> I get this
>
> commit e9f4a0a595eaa82571e1dea55d672ec82da5d2c8
> Author: Mateusz Paprocki <matt...@gmail.com>
> Date: Wed May 4 18:26:49 2011 -0700
>
> Fixed tests after keep_sign was removed
>
> M sympy/core/tests/test_evalf.py
> M sympy/core/tests/test_expr.py
> M sympy/core/tests/test_numbers.py
> M sympy/functions/special/tests/test_gamma_functions.py
> M sympy/functions/special/tests/test_spec_polynomials.py
> M sympy/functions/special/tests/test_spherical_harmonics.py
> M sympy/integrals/tests/test_rationaltools.py
> M sympy/polys/tests/test_partfrac.py
> M sympy/polys/tests/test_polyfuncs.py
> M sympy/polys/tests/test_polyroots.py
> M sympy/series/tests/test_nseries.py
> M sympy/series/tests/test_series.py
> M sympy/simplify/tests/test_simplify.py
> M sympy/solvers/tests/test_ode.py
> M sympy/solvers/tests/test_polysys.py
> M sympy/solvers/tests/test_solvers.py
>
> This looks ugly and files in various subsystems were changed making
> it impossible to infer which subsystem's behaviour was actually
> changed.
>
> Consider a commit that makes several minor changes to sympy/core.
> There are approximately 25 files there and if changes are made to all
> of
> them then it will reflect with the above command. Dont forget the
> tests
> too.
>
> My original point was, that for the purposes of release documentation
> I do *not*
> want to see the code, I am only interested in what it does. Its
> somewhat troublesome
> having to see which files were modified when one can simply specify
> which
> folder's content was changed.
>
> Anyway, bugfixes normally start with
> issue ###: blah blah
> which is clear and concise. The same can be done for non bugfix
> patches.
>
>> --
>> Robert Kern
>>
>> "I have come to believe that the whole world is an enigma, a harmless
>> enigma that is made terrible by our own mad attempt to interpret it as
>> though it had an underlying truth."
>> -- Umberto Eco
> I bought The Black Swan a few days ago, its awesome!
>
> --
> You received this message because you are subscribed to the Google Groups
> "sympy" group.
> To post to this group, send email to sympy@googlegroups.com.
> To unsubscribe from this group, send email to
> sympy+unsubscr...@googlegroups.com.
> For more options, visit this group at
> http://groups.google.com/group/sympy?hl=en.
>
>
--
You received this message because you are subscribed to the Google Groups
"sympy" group.
To post to this group, send email to sympy@googlegroups.com.
To unsubscribe from this group, send email to
sympy+unsubscr...@googlegroups.com.
For more options, visit this group at
http://groups.google.com/group/sympy?hl=en.
