Skip to content
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

add appendix with topic pathways #354

Merged
merged 2 commits into from
Mar 27, 2018
Merged

Conversation

willingc
Copy link
Collaborator

Related to #40, #120, #189

This PR adds an Appendix that groups the individual document pages into logical subtopics. The goal of this PR is to gather topics together to help:

  • consolidation and refactoring of sections that only have one paragraph
  • deduplicate redundant content

My hope is that this can be a first pass and that we can iterate on this in future PRs.

screenshot 2018-03-26 07 47 19

@ezio-melotti
Copy link
Member

This looks great, and might even be a first step in reordering the pages in the devguide (something I wanted to do as part as #120), eventually making this appendix obsolete.

One distinction I want to make that is missing from this, is that contributors should read setup.rst and pullrequest.rst; core devs should read setup.rst, pullrequest.rst, and committing.rst.
All other pages don't necessarily have to be read, but consulted if and when the need arises.

@willingc
Copy link
Collaborator Author

Hi @ezio-melotti. Glad you like it. My goal was to make this a starting point for refactoring/deduplicating content, and I agree it may be unneeded after that.

I've gone ahead and added the sections you mention and put them in a note box. I'm going to go ahead and merge. Feel free to iterate on another PR. Thanks.

@willingc willingc merged commit 4e456f2 into python:master Mar 27, 2018
@willingc willingc deleted the contents-bytype branch March 27, 2018 04:11
AA-Turner pushed a commit to AA-Turner/devguide that referenced this pull request Jun 17, 2022
* add appendix with topic pathways
* add @eziomelotti review feedback
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants