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

Resolved warnings during document generation. #408

Open
wants to merge 10 commits into
base: develop/v2
Choose a base branch
from

Conversation

KanaiYuma-aist
Copy link
Collaborator

ドキュメント生成時に発生していた各種警告文を解消しました。

@KanaiYuma-aist KanaiYuma-aist marked this pull request as draft December 18, 2024 02:07
@KanaiYuma-aist KanaiYuma-aist marked this pull request as ready for review December 18, 2024 02:17
@@ -18,7 +18,7 @@ jobs:
python -m pip install .[all_dev]
- name: Sphinx build
run: |
sphinx-apidoc --maxdepth 2 -f -o ./docs/source/api_reference/ ./aiaccel/
sphinx-apidoc --maxdepth 2 -f -o ./docs/source/api_reference/ -s inc ./aiaccel/
Copy link
Collaborator

Choose a reason for hiding this comment

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

ありがとうございます.こちらですが,workaroundの類なので,あとから意図が分かるよう参照した記事のリンクをコメントで追記下さい.
また,incで名前が良いのか不安です.incが何の略か分かっていないのですが,慣例でしょうか? あまり例がなければ,例えば,*.rst_ にするなど,実態を分かり易くした方が良いかなと思いました.ご検討お願いします.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

開発者会議メモ

  • 拡張子を変更する warning 回避策は削除して、それで表示される警告文に関してはそのままにしておく

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

対応しました

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