Zulip Chat Archive
Stream: general
Topic: Discussions on github for PR, issues etc
Johannes Hölzl (Sep 15 2018 at 16:28):
I think there are a lot of discussions at Zulip, which result in a PR. But often the discussion is missing in the pull request description. I have a hard time to follow all the discussions on Zulip, so I'm often missing why a parch in a PR is done in the specific way. Is it possible to have the discussions witch are PR specific on github itself?
For example for https://github.com/leanprover/mathlib/pull/339 I remember skimming over Zulip and that there was a discussion about the coeff
map instead of using coercion to functions.
The advantage of linking of copying the discussion to the PR itself is that we can later see why a certain change was done. And I try to reference the github PR in the git commit message.
Bryan Gin-ge Chen (Sep 15 2018 at 16:33):
You could add a place in the "TO CONTRIBUTORS" text for links to any relevant zulip thread(s).
Johannes Hölzl (Sep 15 2018 at 16:49):
Oh, good idea!
Scott Morrison (Sep 15 2018 at 23:29):
Links back to Zulip are a start, but Johannes is right that for the longer term conversations archived on github are more useful. Unclear where the balance is!
Last updated: Dec 20 2023 at 11:08 UTC