Status: New
Owner: Vinzent.Steinberg
CC: [email protected], asmeurer
Labels: Type-Enhancement Priority-Medium
New issue 2157 by Vinzent.Steinberg: sympy development rules
http://code.google.com/p/sympy/issues/detail?id=2157
I think there is a consensus about the requirements for inclusion of a
(nontrivial) patch in sympy:
1. All tests have to pass. (Include new tests for new features.)
2. The patch has to be positively reviewed by someone else. (If there a
objections, a consensus has to be reached.)
3. Wait at least 24 hours before pushing it in.
I think it should be easily accessible from the website [1], probably from
the README too. It should be clear to newcomers that not only contribution
is very welcome, but also reviewing patches. (The latter currently lacks
somewhat imho.) People new to sympy should not be afraid to review patches.
Maybe there should even be a short how-to about reviewing patches.
What do you think?
[1] http://sympy.org/development.html
--
You received this message because you are subscribed to the Google Groups
"sympy-issues" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/sympy-issues?hl=en.
