Zulip Chat Archive
Stream: general
Topic: adding to someone else's PR
Kevin Buzzard (May 16 2018 at 20:45):
I have proved that the structure sheaf on an affine scheme is a sheaf of sets. I now need to prove it's a sheaf of commutative rings and we will have our first examples of schemes. However I need a little more transport of structure. In particular I need that the direct limit of commutative rings is a commutative ring. I asked Kenny if he'd done this and he said that he did a big direct limit PR recently but he only did rings, not commutative rings.
Kenny Lau (May 16 2018 at 20:45):
you can PR my PR
Kevin Buzzard (May 16 2018 at 20:45):
I can probably figure out how to add commutative rings myself, but this PR has not yet been accepted by mathlib and rather than having to wait for Mario twice
Kevin Buzzard (May 16 2018 at 20:45):
it seemed sensible just to add it now
Kevin Buzzard (May 16 2018 at 20:46):
How do I PR a PR?
Kevin Buzzard (May 16 2018 at 20:46):
Oh I see -- I send _you_ the PR?
Kevin Buzzard (May 16 2018 at 20:46):
So I fork your mathlib?
Kenny Lau (May 16 2018 at 20:46):
right
Kevin Buzzard (May 16 2018 at 20:46):
OK thanks
Sebastian Ullrich (May 16 2018 at 20:48):
You can directly push to PR branches of repositories you have push access to https://help.github.com/articles/committing-changes-to-a-pull-request-branch-created-from-a-fork/
Kevin Buzzard (May 16 2018 at 20:51):
I'm not sure I have push access to either mathlib or Kenny's fork of it
Kevin Buzzard (May 16 2018 at 23:34):
I can't fork Kenny's mathlib, github complains that I already have some other fork of mathlib
Kevin Buzzard (May 16 2018 at 23:34):
Kenny I've made my edits, I will just email them to you
Kenny Lau (May 16 2018 at 23:35):
ok
Kevin Buzzard (May 16 2018 at 23:35):
I added commutative rings
Last updated: Dec 20 2023 at 11:08 UTC