Closing Old Issues

93 views
Skip to first unread message

Shishir Kushwaha

unread,
Feb 6, 2024, 2:42:44 AM2/6/24
to sympy
I was going over the open issues i could potentially solve and I think some of them are still open after their solution PR has been merged. What do I do about those issues . Should I leave a comment on the issue on its closing or ask what needs to be done ?

Shishir Kushwaha

Oscar Benjamin

unread,
Feb 6, 2024, 7:13:22 AM2/6/24
to sy...@googlegroups.com
On Tue, 6 Feb 2024 at 07:42, Shishir Kushwaha
<kushwahas...@gmail.com> wrote:
>
> I was going over the open issues i could potentially solve and I think some of them are still open after their solution PR has been merged. What do I do about those issues . Should I leave a comment on the issue on its closing or ask what needs to be done ?

Sometimes an issue that should have been closed gets forgotten when a
PR is merged. Also sometimes there are several PRs for the same issue
and once one is merged the others should be closed but this can also
be missed. In either case it is useful to comment on the issue or PR
pointing out that it could be closed.

Please include information showing how to demonstrate that the problem
is fixed and mention that the PR that was merged has a test for the
issue or point to where the issue is tested in the codebase. Basically
include enough information that it is easy for someone else to read
your comment and then confirm that it should be closed.

--
Oscar

Shishir Kushwaha

unread,
Feb 7, 2024, 12:20:45 PM2/7/24
to sympy
There are a few open issues where the solution to the problem of the author has been provided by a contributor or a member , those solutions maybe helpful to others in the future , so do they need to remain open or we make a comment to close them . Do you think it would be better if we added those solutions to a file like a guide ? 

ABHISHEK KUMAR

unread,
Feb 8, 2024, 12:57:25 AM2/8/24
to sy...@googlegroups.com
I was going through old issues and I found two issues in integrals that need to be closed because it was already solved.

These are the links to related issues that needs to be closed.



Please let me know if I should write tests for these issues.

--
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 sympy+un...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/sympy/d3f66066-8741-4825-913b-0deba2168061n%40googlegroups.com.
Reply all
Reply to author
Forward
0 new messages