Skip to content

Conversation

@vsajip
Copy link
Member

@vsajip vsajip commented Aug 19, 2022

No description provided.

@vsajip vsajip requested a review from ezio-melotti August 19, 2022 07:03
for areas of interest to you.)

If you want notification of new issues, you can use the appropriate GitHub notification
settings for the ``python/cpython`` repository.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FWIW I'm planning to add more info about GitHub notifications. I'll take care to update the links once I do.

vsajip and others added 2 commits August 19, 2022 18:24
@vsajip
Copy link
Member Author

vsajip commented Aug 24, 2022

I'm also not sure about the value of mentioning GMane nowadays, though I have left that part unchanged.

@vsajip
Copy link
Member Author

vsajip commented Aug 24, 2022

The built documentation for this PR is (for now) at https://docs.red-dove.com/devguide/ for ease of viewing.

@ezio-melotti
Copy link
Member

It's also available automatically at https://cpython-devguide--938.org.readthedocs.build/ (you can access this by clicking on "Details" next to the readthedocs check).

@vsajip
Copy link
Member Author

vsajip commented Aug 24, 2022

It's also available automatically at ...

Great, I didn't realise that 😄 I removed my built version.

Copy link
Member

@CAM-Gerlach CAM-Gerlach left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great overall; thanks @vsajip . Various typographical, copyediting and syntax suggestions...

vsajip and others added 15 commits August 25, 2022 07:00
Co-authored-by: C.A.M. Gerlach <CAM.Gerlach@Gerlach.CAM>
Co-authored-by: C.A.M. Gerlach <CAM.Gerlach@Gerlach.CAM>
Co-authored-by: C.A.M. Gerlach <CAM.Gerlach@Gerlach.CAM>
Co-authored-by: C.A.M. Gerlach <CAM.Gerlach@Gerlach.CAM>
Co-authored-by: C.A.M. Gerlach <CAM.Gerlach@Gerlach.CAM>
Co-authored-by: C.A.M. Gerlach <CAM.Gerlach@Gerlach.CAM>
Co-authored-by: C.A.M. Gerlach <CAM.Gerlach@Gerlach.CAM>
Co-authored-by: C.A.M. Gerlach <CAM.Gerlach@Gerlach.CAM>
Co-authored-by: C.A.M. Gerlach <CAM.Gerlach@Gerlach.CAM>
Co-authored-by: C.A.M. Gerlach <CAM.Gerlach@Gerlach.CAM>
Co-authored-by: C.A.M. Gerlach <CAM.Gerlach@Gerlach.CAM>
Co-authored-by: C.A.M. Gerlach <CAM.Gerlach@Gerlach.CAM>
Co-authored-by: C.A.M. Gerlach <CAM.Gerlach@Gerlach.CAM>
Co-authored-by: C.A.M. Gerlach <CAM.Gerlach@Gerlach.CAM>
@CAM-Gerlach
Copy link
Member

CAM-Gerlach commented Aug 26, 2022

@vsajip Quick GitHub tip: you can apply all the changes you want in one commit instead of having to make one tiny commit per change, by going to the Files tab and clicking Add to batch on the commits you want, then Commit with your desired message. Also, if you notice a reviewer made a mistake in their suggestion (like I did) or prefer something slightly different, you can reply with your own suggestion and apply that (or edit the original), rather than having to do so manually and then manually mark it as resolved, etc.

Copy link
Member

@CAM-Gerlach CAM-Gerlach left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A couple small followup comments, otherwise LGTM, thanks

can be accessed through the `online archive <web gateway_>`__.

General Python questions should go to `python-list`_ or `tutor`_
or similar resources, such as StackOverflow_ or the ``#python`` IRC channel
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can't reach it, but it might be worth updating the links in the "complete list of Python mailing lists" since it still points to the Mailman 2 instance, when all but a few active lists have been migrated to Mailman 3 by now.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll leave the Mailman2 link there, but add the Mailman3 link.

@vsajip
Copy link
Member Author

vsajip commented Aug 26, 2022

Quick GitHub tip

Ah, right, thanks, I aim to remember this for next time 😄

vsajip and others added 2 commits August 26, 2022 09:40
Copy link
Member

@CAM-Gerlach CAM-Gerlach left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks @vsajip — one side-comment.

@vsajip
Copy link
Member Author

vsajip commented Aug 27, 2022

@ezio-melotti Is it OK to merge this?

Copy link
Member

@ezio-melotti ezio-melotti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants