-
Notifications
You must be signed in to change notification settings - Fork 8
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
Set up doc for Ansys Sphinx theme like PyAnsys libraries #103
Conversation
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.
Some minor changes (especially regarding folder naming conventions). But apart from that all good. Thanks @PipKat!
Regarding Vale, I do agree we should implement it. Let me open a different PR for this 😃
Hi @PipKat! I implemented vale in the Please run |
Incorporate Roberto's feedback Co-authored-by: Roberto Pastor Muela <roberto.pastormuela@ansys.com>
Incorporate missed change. Co-authored-by: Roberto Pastor Muela <roberto.pastormuela@ansys.com>
Google style guide: When referring to the abstract data type, use lowercase. (In the unlikely event that you refer to Boolean mathematics or Boolean logic, use uppercase.)
Instead of the odd sections in the nav bar, I redid this to use Getting Started and User Guide.
Do we want to implement Vale?