Skip to content
This repository has been archived by the owner on May 31, 2020. It is now read-only.

How-To Update for Docs (building on #816) #826

Open
wants to merge 30 commits into
base: master
Choose a base branch
from
Open

How-To Update for Docs (building on #816) #826

wants to merge 30 commits into from

Conversation

pzrq
Copy link
Contributor

@pzrq pzrq commented Aug 6, 2019

This PR builds on #819, specifically it:

  • resolves the merge conflicts with master
  • fixes some typos
  • adds some links, especially to the builtins guide
  • migrates the builtins guide from list() (which I think is a more advanced-level guide, and also covered better in types) to a more intermediate-level guide with the max() function. I'm not wedded to max(), its errors just seemed a little more interesting than abs()

Many of the changes above were suggested as feedback on #816, so I have done what I can to move this forward.

PR Checklist:

  • All new features have been tested
  • All new features have been documented
  • I have read the CONTRIBUTING.md file
  • I will abide by the code of conduct

alexjdw and others added 30 commits June 11, 2019 16:26
… section. Added a tour of the code. Fixed a grammar error.
Conflicts:
	docs/how-to/contribute-docs.rst
	docs/how-to/index.rst

Decided to resolve the `docs/how-to/index.rst` conflict as not including the titles (i.e. follow the Don't Repeat Yourself or DRY principle). A rendered example can be seen at:
https://batavia.readthedocs.io/en/latest/how-to/

This builds on #816 though as noted there it leaves the docs somewhat repetitive and so more completely merging with #823 seems appropriate.
I can see the pythonanywhere.com is at 3.7.0, though CPython is considered somewhat more authoritative. If repl.it (at 3.7.4) allows embedding in future, that may become a better candidate for python.org, though many people still prefer the lower latency of a local machine terminal shell-like experience which the shell version suggests (perhaps even encourages with the higher latency to wait for it to load). Anyway I'm sure someone else is aware of far more detail than I can see here.
They are the star on which this guide is constructed.
From general experience, a lot of developers and engineers I've worked with acknowledge the ECMA specifications, though actually consult the MDN resource when they just want a quick overview of something as it's often simply easier than explaining the long history of the language, which is the goal here - get someone started off, without too much detail.
Please forgive my occasional OCD.
I think list() as a collection type in types/List.js (and a 700+ line implementation) would be better in an advanced guide.
Resolves an error in "make html":
/Users/pzrq/Projects/beeware/batavia/docs/how-to/tour.rst:75:Could not lex literal_block as "javascript". Highlighting skipped.
@pzrq
Copy link
Contributor Author

pzrq commented Aug 6, 2019

Aside: I see a warning when running make html which appears to be fixed upstream in readthedocs/sphinx_rtd_theme#728

(venv) pzrq@mozzie:~/Projects/beeware/batavia/docs$ make html
sphinx-build -b html -W -d _build/doctrees   . _build/html
Running Sphinx v2.1.2
loading pickled environment... done
building [mo]: targets for 0 po files that are out of date
building [html]: targets for 8 source files that are out of date
updating environment: [] 0 added, 8 changed, 0 removed
reading sources... [100%] how-to/types                                          
looking for now-outdated files... none found
pickling environment... done
checking consistency... done
preparing documents... done
writing output... [100%] index                                                  
generating indices... genindex
writing additional pages... search/Users/pzrq/Projects/beeware/venv/lib/python3.7/site-packages/sphinx_rtd_theme/search.html:20: RemovedInSphinx30Warning: To modify script_files in the theme is deprecated. Please insert a <script> tag directly in your theme instead.
  {{ super() }}

copying static files... done
copying extra files... done
dumping search index in English (code: en) ... done
dumping object inventory... done
build succeeded.

The HTML pages are in _build/html.
Copying tabs assets

Build finished. The HTML pages are in _build/html.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants