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

doc/Add breadcrumb info and make general edits #23

Merged
merged 2 commits into from
Feb 8, 2022

Conversation

PipKat
Copy link
Member

@PipKat PipKat commented Feb 3, 2022

No description provided.

@PipKat PipKat requested review from akaszynski, andreas-hilti and Andy-Grigg and removed request for andreas-hilti February 3, 2022 19:33
@PipKat
Copy link
Member Author

PipKat commented Feb 3, 2022

@Andy-Grigg
I'm sure my poor attempt to add breadcrumb information needs your attention!

Copy link
Contributor

@Andy-Grigg Andy-Grigg left a comment

Choose a reason for hiding this comment

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

I have suggested some changes to the breadcrumb explanation. The other changes in this PR all make sense.

Incorporate review comments

Co-authored-by: Andy Grigg <andrew.grigg@ansys.com>
Copy link
Contributor

@akaszynski akaszynski left a comment

Choose a reason for hiding this comment

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

This looks good to me. Thanks for adding it!

@akaszynski akaszynski requested a review from Andy-Grigg February 8, 2022 15:29
@akaszynski akaszynski merged commit 1120bc7 into main Feb 8, 2022
@akaszynski akaszynski deleted the docs/add_breadcrumb_info branch February 8, 2022 15:30
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.

3 participants