Help me understand how to contribute to bugs report / fixing

ag0aep6g anonymous at example.com
Fri Jan 26 21:18:08 UTC 2018


On 01/26/2018 09:43 PM, Fra Mecca wrote:
> Real world case:
> this bug has been reported recently:
> https://issues.dlang.org/show_bug.cgi?id=18288#add_comment
[...]
>  From a quick glance at the phobos repo, I found no mention of this bug 
> in any closed or open PR, just a PR (#6056, bug 18280) on the same file 
> (comparison.d) that probably fixed the issue for bug 18288.

The issue is actually referenced on the page of PR #6056, in the comment 
by dlang-bot. The check marks in the "auto-close" column indicate that 
the issues should have been closed when the PR was merged. But 
apparently only the first one was actually closed. This might be a bug 
in dlang-bot.

I've filed an issue on the bot:
https://github.com/dlang-bots/dlang-bot/issues/175

> What should I do now?
> I am undecided between:
> - commenting on the bug tracker and close the bug
> - link the pr 6056 on the bug tracker
> - leaving it be

When you can point to the PR that fixed an issue, close the issue as 
RESOLVED FIXED and leave a comment pointing to the PR. Something along 
the lines of "Fixed by https://github.com/dlang/phobos/pull/6056".

When you don't know what fixed an issue, you can close as RESOLVED 
WORKSFORME. But make sure that you've taken all the necessary steps to 
reproduce the issue: operating system, compiler flags, etc.

In both cases, it's great when you can make a PR that adds a test to the 
suite, ensuring that the issue doesn't come back. PR #6056 already adds 
the tests (as far as I see), so that's not needed here.


More information about the Digitalmars-d-learn mailing list