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: Add Core Library to introduction section of the Sway book #6193

Merged
merged 20 commits into from
Jul 15, 2024

Conversation

calldelegation
Copy link
Contributor

@calldelegation calldelegation commented Jun 26, 2024

Description

https://fuellabs.slack.com/archives/C031TTYJM60/p1719402172570099

Checklist

  • I have linked to any relevant issues.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have updated the documentation where relevant (API docs, the reference, and the Sway book).
  • I have added tests that prove my fix is effective or that my feature works.
  • I have added (or requested a maintainer to add) the necessary Breaking* or New Feature labels where relevant.
  • I have done my best to ensure that my PR adheres to the Fuel Labs Code Review Standards.
  • I have requested a review from the relevant team or maintainers.

@calldelegation calldelegation requested review from a team as code owners June 26, 2024 22:54
@calldelegation calldelegation self-assigned this Jun 26, 2024
docs/book/src/introduction/core_library.md Outdated Show resolved Hide resolved
docs/book/src/introduction/core_library.md Outdated Show resolved Hide resolved
Copy link
Contributor

@IGI-111 IGI-111 left a comment

Choose a reason for hiding this comment

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

This entire change is a non starter, you should not manually document the standard library when we've explicitly been building forc doc to do so. What should happen is that the documentation for core is hosted in the same way we host the one for std.

If you feel that core is insufficiently documented, you should add some doc comments directly in there.

All that the book should have is a link to the auto-generated documentation, the same it does for std.

docs/book/src/introduction/standard_library.md Outdated Show resolved Hide resolved
Copy link
Member

@kayagokalp kayagokalp left a comment

Choose a reason for hiding this comment

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

I think core is already hosted here. Maybe we may want to improve the doc comments surrounding the stuff from core instead of writing from scratch.

docs/book/src/introduction/core_library.md Outdated Show resolved Hide resolved
docs/book/src/introduction/core_library.md Outdated Show resolved Hide resolved
docs/book/src/introduction/core_library.md Outdated Show resolved Hide resolved
docs/book/src/introduction/core_library.md Outdated Show resolved Hide resolved
@calldelegation calldelegation requested a review from IGI-111 July 12, 2024 06:21
jjcnn
jjcnn previously approved these changes Jul 12, 2024
@calldelegation calldelegation requested a review from IGI-111 July 14, 2024 18:10
@IGI-111 IGI-111 enabled auto-merge (squash) July 15, 2024 07:45
@IGI-111 IGI-111 merged commit ebc2ee6 into master Jul 15, 2024
42 checks passed
@IGI-111 IGI-111 deleted the docs/add-core-library-intro branch July 15, 2024 08:23
esdrubal pushed a commit that referenced this pull request Aug 13, 2024
## Description
https://fuellabs.slack.com/archives/C031TTYJM60/p1719402172570099

## Checklist

- [ ] I have linked to any relevant issues.
- [x] I have commented my code, particularly in hard-to-understand
areas.
- [x] I have updated the documentation where relevant (API docs, the
reference, and the Sway book).
- [x] If my change requires substantial documentation changes, I have
[requested support from the DevRel
team](https://github.com/FuelLabs/devrel-requests/issues/new/choose)
- [ ] I have added tests that prove my fix is effective or that my
feature works.
- [ ] I have added (or requested a maintainer to add) the necessary
`Breaking*` or `New Feature` labels where relevant.
- [x] I have done my best to ensure that my PR adheres to [the Fuel Labs
Code Review
Standards](https://github.com/FuelLabs/rfcs/blob/master/text/code-standards/external-contributors.md).
- [x] I have requested a review from the relevant team or maintainers.

---------

Co-authored-by: IGI-111 <igi-111@protonmail.com>
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