Skip to content

Conversation

@cretz
Copy link
Member

@cretz cretz commented Feb 4, 2022

What was changed

  • Added client impl
    • Supporting new core approach
  • Added docs scaffolding

I have opened this early PR for review. I still have things of course to do such as:

  • Error structure
  • Better documentation
  • Tests

@cretz cretz requested review from Sushisource and bergundy February 4, 2022 23:05
@cretz
Copy link
Member Author

cretz commented Feb 4, 2022

Note, this is not cleanly applying on top of what's in main, so I'll be opening a new PR from a different branch when we're ready for full review

@cretz
Copy link
Member Author

cretz commented Feb 7, 2022

Since there have been no comments here, I am going to close this and make a new branch properly rebased on main.

@cretz cretz closed this Feb 7, 2022
@cretz
Copy link
Member Author

cretz commented Feb 7, 2022

Moved to #6

@cretz cretz deleted the impl2 branch February 7, 2022 16:03
Copy link
Member

@bergundy bergundy left a comment

Choose a reason for hiding this comment

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

I forgot to submit this in progress review.

@@ -0,0 +1,18 @@
.. _api:
Copy link
Member

Choose a reason for hiding this comment

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

Can we use markdown?
I haven't looked into it much but there seems to be a markdown extension for sphinx https://www.sphinx-doc.org/en/master/usage/markdown.html.

The reasoning is that markdown is much more popular and well known but that might not apply in the case of sphinx docs.

I know rst is the standard for python docs so feel free to ignore this comment.

import os
import sys

sys.path.insert(0, os.path.abspath("../"))
Copy link
Member

Choose a reason for hiding this comment

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

This is relative to PWD, use os.path.abspath(os.path.join(os.path.dirname(__file__), '..')) to make it relative to the file.

@@ -0,0 +1,69 @@
# Configuration file for the Sphinx documentation builder.
Copy link
Member

Choose a reason for hiding this comment

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

Are you eventually planning to put this on readthedocs.org?

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