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 sphinx support - core (ref #1385) #1565

Closed
wants to merge 108 commits into from

Conversation

AA-Turner
Copy link
Member

See #2, #1385 for context.

This is the core Sphinx support part, comprising

  • build.py,
  • conf.py,
  • Makefile,
  • pep2rss.py,
  • GitHub actions,
  • requirements.txt

Note: this is unlikely to successfully build, I am splitting into logical chunks as per Victor's comment.

Co-authored-by: Pradyun Gedam <3275593+pradyunsg@users.noreply.github.com>
@AA-Turner AA-Turner mentioned this pull request Apr 20, 2021
@AA-Turner AA-Turner closed this Apr 20, 2021
@AA-Turner AA-Turner mentioned this pull request Apr 20, 2021
pablogsal pushed a commit that referenced this pull request Jun 8, 2021
See #2, #1385 for context. Superseeds #1565.

This is the minimal core Sphinx support part, adding a bare minimum of useful things to get Sphinx to build and deploy, whilst not affecting the current build system. There is no theming or custom parsing needed to properly deal with PEPs.

- `build.py` - build script
- `conf.py` - Sphinx configuration
- `Makefile` - new targets for Sphinx
- `.gitignore` - add ignores for `venv` and `package` directories
- `contents.rst` - Sphinx page to discover all PEPs
- `deploy-gh-pages.yaml` - builds and deploys to github pages
- `requirements.txt`
pablogsal pushed a commit that referenced this pull request Jun 15, 2021
See #2, #1385 for context.

This is the RSS generation part, building on PR #1930. It contains the logic for generating RSS

This was originally in #1385 and #1565, split out for ease of review
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