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

Improve doc: update optional installation dependencies #13

Merged
merged 13 commits into from
Aug 27, 2024
Merged

Conversation

yiranwu0
Copy link
Collaborator

@yiranwu0 yiranwu0 commented Aug 25, 2024

Why are these changes needed?

Improve optional dependency document.

Related issue number

Checks

Copy link
Collaborator

@XHMY XHMY left a comment

Choose a reason for hiding this comment

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

looks good

Copy link
Collaborator

@marklysze marklysze left a comment

Choose a reason for hiding this comment

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

Tested, looks good, thanks @yiranwu0

Copy link
Collaborator

@Hk669 Hk669 left a comment

Choose a reason for hiding this comment

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

looks good to me

website/docs/installation/Optional-Dependencies.md Outdated Show resolved Hide resolved
website/docs/installation/Optional-Dependencies.md Outdated Show resolved Hide resolved
website/docs/topics/groupchat/resuming_groupchat.mdx Outdated Show resolved Hide resolved
@yiranwu0 yiranwu0 requested a review from sonichi August 26, 2024 18:53
@yiranwu0
Copy link
Collaborator Author

@sonichi Some of the notebooks are not compiled. I can make a separate PR to add metadata to these files and change corresponding references. Do you think it is necessary?

Copy link
Collaborator

@sonichi sonichi left a comment

Choose a reason for hiding this comment

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

Looks good now. No need to make another PR.

@sonichi sonichi merged commit d21695a into main Aug 27, 2024
17 checks passed
@sonichi sonichi deleted the webupdate branch August 27, 2024 13:46
odoochain pushed a commit to odoochain/autogen that referenced this pull request Nov 10, 2024
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.

5 participants