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

docs: adding intro to sway #55

Merged
merged 98 commits into from
Nov 29, 2023
Merged

docs: adding intro to sway #55

merged 98 commits into from
Nov 29, 2023

Conversation

calldelegation
Copy link
Contributor

Taking the readme upgrading to beta-4 and improving clarity as well as adding a rust testing section

@calldelegation
Copy link
Contributor Author

TODO add intro-to-sway as submodule to pull code in. Instead of copy and pasting.

@vercel
Copy link

vercel bot commented Oct 3, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
docs-hub ❌ Failed (Inspect) Oct 3, 2023 9:57pm

Copy link
Contributor

@sarahschwartz sarahschwartz left a comment

Choose a reason for hiding this comment

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

almost there, just a few more nits

docs/guides/docs/guides.json Outdated Show resolved Hide resolved
docs/guides/docs/guides.json Outdated Show resolved Hide resolved
docs/guides/docs/intro-to-sway/index.mdx Show resolved Hide resolved
docs/guides/docs/intro-to-sway/checkpoint.mdx Outdated Show resolved Hide resolved
sarahschwartz
sarahschwartz previously approved these changes Nov 21, 2023
Copy link
Contributor

@sarahschwartz sarahschwartz left a comment

Choose a reason for hiding this comment

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

LGTM, great job!! 👏

@calldelegation calldelegation requested review from cold-briu and removed request for cold-briu November 21, 2023 16:25
Copy link
Contributor

@cold-briu cold-briu left a comment

Choose a reason for hiding this comment

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

Great job!
Thanks for remove the $ characters.

I've dropped a couple of non-blocking comments.

docs/about-fuel/networks/beta-3.md Outdated Show resolved Hide resolved
docs/guides/docs/installation/index.mdx Show resolved Hide resolved
@calldelegation calldelegation merged commit 098cb54 into master Nov 29, 2023
10 checks passed
@calldelegation calldelegation deleted the add/intro-to-sway branch November 29, 2023 01:33
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