-
-
Notifications
You must be signed in to change notification settings - Fork 944
Make updates relating to communication channels. #938
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
core-developers/responsibilities.rst
Outdated
| 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. |
There was a problem hiding this comment.
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.
Co-authored-by: Ezio Melotti <ezio.melotti@gmail.com>
|
I'm also not sure about the value of mentioning GMane nowadays, though I have left that part unchanged. |
|
The built documentation for this PR is (for now) at https://docs.red-dove.com/devguide/ for ease of viewing. |
|
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). |
Great, I didn't realise that 😄 I removed my built version. |
CAM-Gerlach
left a comment
There was a problem hiding this 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...
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>
|
@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. |
CAM-Gerlach
left a comment
There was a problem hiding this 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 |
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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.
Ah, right, thanks, I aim to remember this for next time 😄 |
Co-authored-by: C.A.M. Gerlach <CAM.Gerlach@Gerlach.CAM>
CAM-Gerlach
left a comment
There was a problem hiding this 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.
|
@ezio-melotti Is it OK to merge this? |
ezio-melotti
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
No description provided.