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 sitemap.xml to sphinx docs #1890

Merged
merged 2 commits into from
May 7, 2018
Merged

add sitemap.xml to sphinx docs #1890

merged 2 commits into from
May 7, 2018

Conversation

orbeckst
Copy link
Member

@orbeckst orbeckst commented May 6, 2018

@orbeckst
Copy link
Member Author

orbeckst commented May 7, 2018

rebased against develop, will merge/rebase after statuses.

@orbeckst
Copy link
Member Author

orbeckst commented May 7, 2018

@richardjgowers I decided to generated sitemap.xml for the develop site so that we are guaranteed to have an appropriate sitemap.xml when deploying from travis. We deploy the master docs manually anyway so I added the new adapt_sitemap.py script into the workflow.

The script is doing a search and replace on the XML tree. A simple sed would have done the same thing but is much more fragile.

orbeckst added 2 commits May 7, 2018 16:14
- install https://github.com/jdillard/sphinx-sitemap with pip
- create sitemap.xml that is correct for the RELEASE docs
  (it is incorrect for the development docs but we do not want to
  index the devdocs anyway... but we could change it so that by default
  it is correct for devdocs and then we fix sitemap.xml when we manually
  add the release docs with the maintainer/deploy_master_docs.sh script)
- see MDAnalysis/MDAnalysis.github.io#78
…ment

- sitemap.xml is appropriate for the development docs: thus we do not ship
  and install a sitemap.xml that is misleading (when installing automatically
  via Travis CI)
- change the loc in the sitemap.xml with the new maintainer/adapt_sitemap.py
  script: parse XML tree and replace the developer site URL prefix with the
  release doc prefix
- manually create the release docs from master with the
  maintainer/deploy_master_docs.sh script (which is now calling
  the adapt_sitemap.py script)
@orbeckst
Copy link
Member Author

orbeckst commented May 7, 2018

rebased because of #1892

@orbeckst orbeckst merged commit 8bab359 into develop May 7, 2018
@orbeckst orbeckst deleted the sitemap branch May 7, 2018 23:52
@orbeckst
Copy link
Member Author

orbeckst commented May 8, 2018

I manually ran the doc building for master (with the new sitemap stuff temporarily patched in) and pushed new release docs for 0.18.0 that also contain a sitemap https://www.mdanalysis.org/docs/sitemap.xml

@orbeckst orbeckst mentioned this pull request May 8, 2018
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.

2 participants