It does technically fulfill the requirement, but I would suggest doing something more to make your proposal stronger.
Aaron Meurer On Fri, Mar 24, 2017 at 2:58 AM, Vedarth Sharma <[email protected]> wrote: > Done. And it got merged as well :). Does that fulfill my patch requirement or > do i need to submit a pull request related to my proposal as well? > > -- > 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 https://groups.google.com/group/sympy. > To view this discussion on the web visit > https://groups.google.com/d/msgid/sympy/f01314d0-345e-4dbe-90e4-12861ad182ff%40googlegroups.com. > For more options, visit https://groups.google.com/d/optout. -- 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 https://groups.google.com/group/sympy. To view this discussion on the web visit https://groups.google.com/d/msgid/sympy/CAKgW%3D6Kb17-EW1ay7AwwQOPs%2B%3DzHqH8hZ%2Bmo7oAittpzESongg%40mail.gmail.com. For more options, visit https://groups.google.com/d/optout.
