Zulip Chat Archive
Stream: mathlib4
Topic: label after-port
Yury G. Kudryashov (Feb 03 2023 at 01:26):
While porting, I notice various APIs/definitions that can be improved. Before the port started, I would procrastinate, then create a PR with a refactor. Now, it is not a good time for refactors but I don't want to forget about these ideas (even if half of them will never be implemented). So, I'm going to create a bunch of issues labeled after-port
. Any suggestions/objections?
Mario Carneiro (Feb 03 2023 at 01:58):
Seems fine to me. Should they be opened against mathlib3 or mathlib4?
Yury G. Kudryashov (Feb 03 2023 at 02:00):
mathlib4
Yury G. Kudryashov (Feb 03 2023 at 02:00):
Because after the port, mathlib3 will be frozen.
Eric Wieser (Feb 03 2023 at 02:17):
Issues can be moved between repos anyway
Yury G. Kudryashov (Feb 03 2023 at 02:37):
I want to be able to ref. them by numbers and don't loose these refs after the port (also, I'm trying to stop myself from working on the Lean 3 repo).
Eric Wieser (Feb 03 2023 at 02:38):
I think if you move an issue the old url redirects
Eric Wieser (Feb 03 2023 at 02:39):
So whatever you pick, the refs won't be lost
Yury G. Kudryashov (Feb 03 2023 at 02:39):
Anyway, I already created a few of them in the mathlib4
repo.
Eric Wieser (Feb 03 2023 at 02:40):
On a related note, we should probably go though mathlib 4 and qualify all references to mathlib 3 GitHub issues
Mario Carneiro (Feb 03 2023 at 02:41):
oh you mean in (ported) comments?
Mario Carneiro (Feb 03 2023 at 02:42):
probably we should go through mathlib3 and do that; once it gets in mathlib4 it's hard to differentiate from mathlib4 issue references
Eric Wieser (Feb 03 2023 at 02:42):
Yeah, any occurrence of #NNNN
or gh-NNNN
in a comment should probably get a mathlib3 prefix
Johan Commelin (Feb 03 2023 at 02:43):
Except for NNNN = 2003
:wink:
Eric Wieser (Feb 03 2023 at 02:43):
Mario Carneiro said:
probably we should go through mathlib3 and do that; once it gets in mathlib4 it's hard to differentiate from mathlib4 issue references
We have a rapidly closing window where the numbers are mostly far enough apart to get away with it
Mario Carneiro (Feb 03 2023 at 02:44):
we should hopefully be qualifying lean4 PR numbers already
Last updated: Dec 20 2023 at 11:08 UTC