-
Notifications
You must be signed in to change notification settings - Fork 8
Combine all styles #404
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
Combine all styles #404
Conversation
Review from @PipKat Co-authored-by: Kathy Pippert <84872299+PipKat@users.noreply.github.com>
Co-authored-by: Kathy Pippert <84872299+PipKat@users.noreply.github.com>
|
These are breaking changes. Before merging we should inform everybody. Links will start to fail all around our docs. |
My bad - it won't break anything. Each subsection will still be at the same URL since you didn't move the files. You just created a new one. All good from my side! |
@RobPasMue Not that I've had any time to move content from the PyAnsys writer's guide into the dev guide, but if I did, I feel like I'd need a strategic meeting with you to discuss how best to integrate it. Chris H. suggested reorganizing the dev guide as I do the integration, but I'm sure that would result in broken links. |
Yep, we should have a meeting prior to that taking place because it would break many people's repos. Let's talk offline! :) |
* combine all style to one page * Apply suggestions from code review Review from @PipKat Co-authored-by: Kathy Pippert <84872299+PipKat@users.noreply.github.com> * adjusting bullet list * header links are only 4 so no longer needed * Update doc/source/all-styles.rst Co-authored-by: Kathy Pippert <84872299+PipKat@users.noreply.github.com> * removing spacing between bulletlist * adding hyperlink * reformatting --------- Co-authored-by: Kathy Pippert <84872299+PipKat@users.noreply.github.com>

Combine packaging, coding and documentation styles under Style section