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