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

Document how dep works #735

Merged
merged 9 commits into from
Feb 4, 2022
Merged

Document how dep works #735

merged 9 commits into from
Feb 4, 2022

Conversation

ControlCplusControlV
Copy link
Contributor

Tried to explain dep using examples, although actual definition may still be too short as I've only learned what it does via the standard and core libraries, so feel free to suggest any changes!

docs/src/sway-on-chain/libraries.md Outdated Show resolved Hide resolved
docs/src/sway-on-chain/libraries.md Outdated Show resolved Hide resolved
docs/src/sway-on-chain/libraries.md Outdated Show resolved Hide resolved
docs/src/sway-on-chain/libraries.md Outdated Show resolved Hide resolved
docs/src/sway-on-chain/libraries.md Outdated Show resolved Hide resolved
@mohammadfawaz
Copy link
Contributor

Thank you for doing this!

Will you also please remove the last sentence in this file:
"Note that multiple imports are not yet supported: #563."
The full Rust syntax for imports is now supported.

ControlCplusControlV and others added 6 commits February 2, 2022 09:28
Co-authored-by: Mohammad Fawaz <mohammadfawaz89@gmail.com>
Co-authored-by: Mohammad Fawaz <mohammadfawaz89@gmail.com>
Co-authored-by: Mohammad Fawaz <mohammadfawaz89@gmail.com>
Co-authored-by: Mohammad Fawaz <mohammadfawaz89@gmail.com>
Co-authored-by: Mohammad Fawaz <mohammadfawaz89@gmail.com>
@ControlCplusControlV
Copy link
Contributor Author

Added changes and update that last line

adlerjohn
adlerjohn previously approved these changes Feb 2, 2022
Copy link
Contributor

@sezna sezna 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 -- just one small change!

docs/src/sway-on-chain/libraries.md Outdated Show resolved Hide resolved
Co-authored-by: Alex Hansen <alex@alex-hansen.com>
@ControlCplusControlV
Copy link
Contributor Author

Added!

@adlerjohn adlerjohn requested a review from sezna February 4, 2022 14:27
@sezna sezna merged commit f0707fc into FuelLabs:master Feb 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
The Sway Book Everything to do with the Sway Book
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Document the use of the dep keyword in the Sway book
4 participants