Luke wrote:
> So this was my first patch, I hope I did everything correctly.  I
> added tests for issue 1098 to make sure that
> 
> (x * m/s).integrate((x, 1*s, 5*s)) == integrate(x*m/s,(x, 1*s, 5*s))
> 
> On my first commit I accidentally left some extra print statements in
> the test_units.py, so I created another commit with them removed.  Is
> the general practice to make two separate commits, or can one edit
> things and then add them to an already completed commit?

you can easily squash two commits with git, try:

git rebase -i origin

or google for "git rebase"

> 
> Also, it seems as though I am unable to change the status of Issue
> 1098 on the wiki.  Are issues only able to be close by the originator
> of the issue?  If not, what do I need to be able close an issue?

sounds strange ...

> 
> Feedback welcome.
> 
> Thanks,
> ~Luke
> 
> > 
> 


--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"sympy-patches" group.
To post to this group, send email to sympy-patches@googlegroups.com
To unsubscribe from this group, send email to 
sympy-patches+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/sympy-patches?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to