-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
Update install.rst #5858
Update install.rst #5858
Conversation
minor cleanups/corrections
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.
Thanks for the fixes, LGTM
Looks like one of the edits on the section titles is breaking a link in the RST. Can you fix line 23 of the file to add the missing "VTA "? Thanks |
Until then we cannot merge the fixes. |
@badenh Please let us know if you can fix the install.rst to pass the CI tests |
Fixed broken link
Yes, fixed the missing item now |
thank you @badenh your changes have been merged |
* Update install.rst minor cleanups/corrections * Update install.rst Fixed broken link
* Update install.rst minor cleanups/corrections * Update install.rst Fixed broken link
minor cleanups/corrections
Thanks for contributing to TVM! Please refer to guideline https://tvm.apache.org/docs/contribute/ for useful information and tips. After the pull request is submitted, please request code reviews from Reviewers by @ them in the pull request thread.