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

Overhaul Style Guide #2838

Merged
merged 39 commits into from
Dec 8, 2016
Merged

Overhaul Style Guide #2838

merged 39 commits into from
Dec 8, 2016

Conversation

stevepiercy
Copy link
Member

This PR addresses issue #2633, and expands the Style Guide as an easy reference for documentation authors.

Request review and feedback.

- add how to update and contribute
- add file conventions
- add line conventions
Rename Typographical Conventions to Style Guide
- trailing white space
- indentation
- split line lengths and paragraphs
Add Introduction and other sections
- add description
- add introduction
- add headings
- add grammar, spelling, capitalization preferences
- add literal includes
- add displaying long commands
- add sphinx to intersphinx
- garden
- start work on inline and block reST sections
- add rendering of headings for testing
- add topic directive
- dedent displaying code
- add italics, strong
- add Pytyon modules, classes, methods, functions
- add tables heading (WIP)
- add API documenation (WIP)
- add lists, tables, danger, todo, comments
add todo Sphinx extension for todo blocks
- add parsed-literals
Add more sections to the Style Guide
restore Typographical Conventions, an edited and condensed version of…
@stevepiercy
Copy link
Member Author

@bertjwregeer Typo Conv is back and better than ever.

One day, when we have a useful pylonsproject.org, I'll move the Style Guide over there to share across all Pylons Project projects.

@stevepiercy
Copy link
Member Author

We're close to releasing the new pylonsproject.org. However, I want to use Sphinx to render style-guide.rst because it is for docs and uses Sphinx markup, so it makes sense to have it here and rendered.

If there are no further suggestions or objections, then I'll merge this PR this evening.

@digitalresistor
Copy link
Member

There are things in the Typographical Conventions that don't belong there.

Such as what a bulleted list is, or headings. It needs to contain a little information for users to know "Ah, that means it's code" or "that is a command I run" or "this is a warning".

Here's some typographical conventions from three random books I pulled down off my bookshelf:

For Dummies:
fullsizerender 3

Cisco Press:
fullsizerender 2

O'Reilly:
fullsizerender 4

@stevepiercy
Copy link
Member Author

@bertjwregeer Updated. I'll merge this evening if there are no further suggestions.

@digitalresistor
Copy link
Member

👍 much better. Thanks!

@stevepiercy
Copy link
Member Author

Thanks for the feedback!

@stevepiercy stevepiercy merged commit 05467bd into master Dec 8, 2016
@stevepiercy stevepiercy deleted the style-guide branch December 8, 2016 23:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants