Usually, when a PR needs fixing, it is done in the same PR, and there may be need to rebase so that commit history is not polluted, hence git push -f <br><div class="gmail_quote">On Tue, Jun 9, 2015 at 3:59 PM Lennart Poettering <<a href="mailto:lennart@poettering.net">lennart@poettering.net</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Tue, 09.06.15 18:42, David Timothy Strauss (<a href="mailto:david@davidstrauss.net" target="_blank">david@davidstrauss.net</a>) wrote:<br>
<br>
> Let's just try the GitHub tracker. I like how it associates issues with<br>
> pull requests and supports auto-linking for commit IDs, user names, and<br>
> other issue numbers. Is there any serious use case for systemd upstream it<br>
> doesn't support?<br>
<br>
Here's one issue I have with the github bug tracker. Or maybe it's<br>
more an issue with the github PR tracker... Anyway, if somebody finds<br>
a bug in systemd two things might happen:<br>
<br>
     a) if he's a user/admin without C skills, he'll just file an<br>
        issue and that's it. that's great.<br>
<br>
     b) if he's a hacker with C skills he'll likely send us a PR<br>
        instead. But this is where the problems now start: Most likely<br>
        the first patch iteration will not be right, so we comment and<br>
        ask for a new PR, and close the old one. The two PRs will not<br>
        be closely related to each other then. In the best case they<br>
        will "reference" each other, but that's it. However what's<br>
        worse is that the bug is not tracked at all in the the time<br>
        between the old PR got closed and the new is opened. Now, we<br>
        could require that in the meantime a new "issue" has to be<br>
        created, but this is a manual process, and it will not inherit<br>
        labels and stuff, unless the user does so manually. And that's<br>
        nasty.<br>
<br>
And of course, the issue that in case of a) we only end up in an<br>
issue, and in case of b) when the first patch is actually good we only<br>
end up in a PR is a major issue too.<br>
<br>
I am not sure what to do about this yet. Maybe we can enforce that PRs<br>
have to have an issue, or so.<br>
<br>
Maybe anyone has an idea?<br>
<br>
Lennart<br>
<br>
--<br>
Lennart Poettering, Red Hat<br>
_______________________________________________<br>
systemd-devel mailing list<br>
<a href="mailto:systemd-devel@lists.freedesktop.org" target="_blank">systemd-devel@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/systemd-devel" target="_blank">http://lists.freedesktop.org/mailman/listinfo/systemd-devel</a><br>
</blockquote></div>