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.

Sign up for GitHub

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

Jump to bottom

Swarm Documentation #28

Merged
merged 16 commits into from
Nov 21, 2024
Merged

Swarm Documentation #28

merged 16 commits into from
Nov 21, 2024

Conversation

yiranwu0
Copy link
Collaborator

Swarm Documentation

Why are these changes needed?

Related issue number

Checks

@marklysze marklysze added the documentation Improvements or additions to documentation label Nov 17, 2024
sonichi
sonichi reviewed Nov 18, 2024
View reviewed changes
website/blog/2024-11-17-Swarm/index.mdx Outdated Show resolved Hide resolved
website/blog/2024-11-17-Swarm/index.mdx Outdated Show resolved Hide resolved
yanenli2
yanenli2 reviewed Nov 19, 2024
View reviewed changes
Copy link
Collaborator

@yanenli2 yanenli2 left a comment

Choose a reason for hiding this comment

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

Leave some clarification questions.

sonichi
sonichi reviewed Nov 20, 2024
View reviewed changes
website/blog/2024-11-17-Swarm/index.mdx Outdated Show resolved Hide resolved
website/blog/2024-11-17-Swarm/index.mdx Outdated Show resolved Hide resolved
website/blog/2024-11-17-Swarm/index.mdx Outdated Show resolved Hide resolved
website/blog/2024-11-17-Swarm/index.mdx Outdated Show resolved Hide resolved
@sonichi sonichi linked an issue Nov 20, 2024 that may be closed by this pull request
[Feature Request]: new high-level programming interface #26
Open
@sonichi sonichi requested a review from pipinstallyp November 20, 2024 10:52
sonichi
sonichi reviewed Nov 20, 2024
View reviewed changes
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.

I have some ideas for research.

website/blog/2024-11-17-Swarm/index.mdx Outdated Show resolved Hide resolved
website/blog/2024-11-17-Swarm/index.mdx Show resolved Hide resolved
@yiranwu0 yiranwu0 mentioned this pull request Nov 20, 2024
Open
@yiranwu0 yiranwu0 marked this pull request as ready for review November 20, 2024 21:20
@yiranwu0 yiranwu0 requested a review from yanenli2 November 20, 2024 21:23
@yiranwu0
Copy link
Collaborator Author

@yanenli2 Thank you for your comments. We updated the inferfaces and documents in the past few days and it is quiet different now. Please take a look if you have time. Thank you!

qingyun-wu
qingyun-wu approved these changes Nov 21, 2024
View reviewed changes
Copy link
Collaborator

@qingyun-wu qingyun-wu 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. Thank you Yiran and Mark.

@qingyun-wu qingyun-wu merged commit c247bd0 into main Nov 21, 2024
17 checks passed
@marklysze marklysze deleted the swarmdoc branch November 26, 2024 18:34
davorrunje pushed a commit that referenced this pull request Dec 19, 2024
@harishmohanraj
Merge pull request #28 from harishmohanraj/fix-internal-links
b92342b 
Fix internal links
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sonichi sonichi sonichi left review comments

@qingyun-wu qingyun-wu qingyun-wu approved these changes

@marklysze marklysze Awaiting requested review from marklysze

@pipinstallyp pipinstallyp Awaiting requested review from pipinstallyp

@yanenli2 yanenli2 Awaiting requested review from yanenli2

Assignees
No one assigned
Labels
documentation Improvements or additions to documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Feature Request]: new high-level programming interface
5 participants
@yiranwu0 @sonichi @qingyun-wu @yanenli2 @marklysze