Well, it's always easier for us to merge a change if there's a PR. Otherwise we have to download the patch, apply it, run the tests, and push it in. With a pull request, the tests are run automatically by Travis, and if there's a green light, we can merge it with a single button.
Aaron Meurer On Sat, Mar 2, 2013 at 2:07 PM, Dale Lukas Peterson <[email protected]> wrote: > sympy/printing/codeprinter.py:175 uses S.One, but S has not been > imported. The attached patch fixes this. > > I can open a PR if needed but this is such a minor change I figured it > might not be necessary. > > Luke > > > -- > "People call me a perfectionist, but I'm not. I'm a rightist. I do > something until it's right, and then I move on to the next thing." > -- James Cameron > > -- > You received this message because you are subscribed to the Google Groups > "sympy" group. > To unsubscribe from this group and stop receiving emails from it, send an > email to [email protected]. > To post to this group, send email to [email protected]. > Visit this group at http://groups.google.com/group/sympy?hl=en. > For more options, visit https://groups.google.com/groups/opt_out. > > -- You received this message because you are subscribed to the Google Groups "sympy" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To post to this group, send email to [email protected]. Visit this group at http://groups.google.com/group/sympy?hl=en. For more options, visit https://groups.google.com/groups/opt_out.
