Skip to content

[Doc] Fix incorrect command#11630

Merged
driazati merged 2 commits intoapache:mainfrom
huajsj:doc
Jun 21, 2022
Merged

[Doc] Fix incorrect command#11630
driazati merged 2 commits intoapache:mainfrom
huajsj:doc

Conversation

@huajsj
Copy link
Contributor

@huajsj huajsj commented Jun 8, 2022

build/html get moved into _docs, update related document.

build/html get moved into _docs, update related document.
@huajsj
Copy link
Contributor Author

huajsj commented Jun 14, 2022

@driazati

@driazati
Copy link
Member

the docs only get moved as part of task_python_docs.sh it looks like, did you see them moved after just running make html (which afaik we haven't customized at all so it just does the standard sphinx thing)? for running python tests/scripts/ci.py docs the folder does get moved but python tests/scripts/ci.py serve-docs should handle that

rm -rf _docs
mv docs/_build/html _docs

@huajsj
Copy link
Contributor Author

huajsj commented Jun 14, 2022

the docs only get moved as part of task_python_docs.sh it looks like, did you see them moved after just running make html (which afaik we haven't customized at all so it just does the standard sphinx thing)? for running python tests/scripts/ci.py docs the folder does get moved but python tests/scripts/ci.py serve-docs should handle that

rm -rf _docs
mv docs/_build/html _docs

you are right, make html do not move the file, we should still use the path _build/html for native
fixed another issue that before "make html", should first go into "docs", the existing guide not mention this.

@huajsj
Copy link
Contributor Author

huajsj commented Jun 20, 2022

@driazati, could help for the patch merge?

@driazati driazati merged commit 9e58af2 into apache:main Jun 21, 2022
blackkker pushed a commit to blackkker/tvm that referenced this pull request Jul 7, 2022
build/html get moved into _docs, update related document.
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