Skip to content

docs: rework documentation#238

Merged
andreiltd merged 11 commits into
bytecodealliance:mainfrom
andreiltd:docs
Jul 22, 2025
Merged

docs: rework documentation#238
andreiltd merged 11 commits into
bytecodealliance:mainfrom
andreiltd:docs

Conversation

@andreiltd
Copy link
Copy Markdown
Member

@andreiltd andreiltd commented Apr 14, 2025

This patch splits documentation into multiple markdown files and deploys mdbook to github pages. There are also new sections like developing builtins in c++.

The rendered mdbook can be found here: https://andreiltd.github.io/StarlingMonkey/

@andreiltd andreiltd marked this pull request as draft April 14, 2025 11:12
@andreiltd andreiltd force-pushed the docs branch 29 times, most recently from e7bf7b7 to 1e2fd03 Compare April 14, 2025 17:42
@andreiltd andreiltd requested a review from tschneidereit April 25, 2025 16:46
Copy link
Copy Markdown
Contributor

@vados-cosmonic vados-cosmonic left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

I have a bunch of comments but none of them are blocking -- the value of this improvement in documentation is massive. Happy to have this PR go in as-is, but felt I should at least leave some nits worth thinking about.

Comment thread .github/workflows/docs.yml Outdated
Comment thread .github/workflows/docs.yml Outdated
Comment thread .github/workflows/docs.yml
Comment thread .github/workflows/docs.yml
Comment thread .github/workflows/docs.yml Outdated
Comment thread docs/src/developer/spidermonkey.md Outdated
Comment thread docs/src/developer/spidermonkey.md Outdated
Comment thread docs/src/developer/spidermonkey.md Outdated
Comment thread docs/src/resources/community.md
Comment thread docs/src/resources/community.md Outdated
@andreiltd andreiltd force-pushed the docs branch 13 times, most recently from dcb3c34 to 3e68a4b Compare July 15, 2025 12:41
Copy link
Copy Markdown
Member

@tschneidereit tschneidereit left a comment

Choose a reason for hiding this comment

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

This is really really fantastic, thank you so much!

I left a bunch of comments and suggestions, but none of them should hold up landing this whenever you feel ready to do so.

Comment thread docs/src/developer/builtins-cpp.md Outdated
Comment thread docs/src/developer/builtins-cpp.md Outdated
Comment thread docs/src/developer/builtins-cpp.md Outdated
Comment thread docs/src/developer/builtins-cpp.md Outdated
Comment thread docs/src/developer/builtins-cpp.md Outdated
Comment thread docs/src/getting-started/building.md Outdated
Comment thread docs/src/getting-started/building.md Outdated
Comment thread docs/src/getting-started/building.md Outdated
Comment thread docs/src/getting-started/building.md Outdated
Comment thread docs/src/getting-started/building.md Outdated
andreiltd and others added 2 commits July 22, 2025 11:35
Co-authored-by: Till Schneidereit <till@tillschneidereit.net>
@andreiltd andreiltd changed the title Rework documentation docs: rework documentation Jul 22, 2025
@andreiltd andreiltd merged commit 3850bd8 into bytecodealliance:main Jul 22, 2025
7 checks passed
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