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 Python API reference docs #1358
Conversation
"keypair" should really be "key pair" in docstrings and such but is written as keyPair in Nix expressions. Codespell says "IAM" should be "I am" which is incorrect (IAM is an AWS thing). Codespell says "hda" should be "had" which is wrong for obvious reasons :)
Reference docs surfaces a lot of issues with circular import. Let's not block reference docs on fixing circular imports, this is another issue entirely.
ef2c411
to
909c18e
Compare
@@ -3,6 +3,6 @@ | |||
|
|||
set -eux | |||
|
|||
git ls-files | xargs codespell | |||
git ls-files | xargs codespell -L keypair,iam,hda |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is keypair
keyword still needed here since there are fixed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, the nix expressions still says keyPair
.
sphinx-build -M clean doc/ doc/_build | ||
sphinx-build -nW doc/ doc/_build | ||
sphinx-build -n doc/ doc/_build |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we really need to remove W
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes :/
We have circular imports guarded by typing.TYPE_CHECKING
. That's not a problem for Mypy but it is a problem for Sphinx.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok :(
No description provided.