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

chore: sphinx documentation configs, build script, CI job #117

Merged
merged 1 commit into from
Aug 4, 2021

Conversation

lost-theory
Copy link
Contributor

What I did

Related issue: #21

How I did it

I used some of the sphinx setup I did in eip712, but added quite a bit of work on top of that, including the sphinx-autoapi plugin so we don't have to generate or maintain .rst files for full API documentation of modules, classes, and functions.

How to verify it

A new CI job for docs will on PRs, branches, and releases.

Checklist

  • Passes all linting checks (pre-commit and CI jobs)
  • New test cases have been added and are passing
  • Documentation has been updated
  • PR title follows Conventional Commit standard (will be automatically included in the changelog)

src/ape/types/__init__.py Outdated Show resolved Hide resolved
src/ape/managers/networks.py Outdated Show resolved Hide resolved
@@ -0,0 +1,69 @@
Quickstart guide
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wonder if there is a way to pull this content from the README.md?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We could add a step to build_docs.py to do that. It would make things easier if we converted it from markdown to ReST.

src/ape/__init__.py Outdated Show resolved Hide resolved
@lost-theory lost-theory force-pushed the chore/docs branch 4 times, most recently from 6b3602b to aa0e452 Compare August 4, 2021 16:27
@fubuloubu fubuloubu enabled auto-merge (squash) August 4, 2021 16:30
@fubuloubu fubuloubu merged commit aa8cc4d into ApeWorX:main Aug 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants