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 WSL installation guide #946

Merged
merged 1 commit into from
Dec 10, 2024
Merged

Conversation

ileana-pr
Copy link
Contributor

Relates to:

AI Agent Dev School Video 1 : https://www.youtube.com/watch?v=ArptLpQiKfI

Risks

Low

Background

Shaw suggests using WSL to code with Eliza but there are some steps missing from the instructions in order for installation to work.

What does this PR do?

Fixes pnpm installation error when using WSL for the first time.

What kind of change is this?

Documentation

Why are we doing this? Any context or related work?

Documentation changes needed?

My changes require a change to the project documentation.

Testing

  • Verified all commands work in a fresh WSL installation
  • Confirmed formatting matches existing documentation

Where should a reviewer start?

Video:
https://www.youtube.com/watch?v=ArptLpQiKfI

Detailed testing steps

none

Copy link
Collaborator

@monilpat monilpat left a comment

Choose a reason for hiding this comment

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

LGTM!

@jkbrooks jkbrooks merged commit 9c7a080 into elizaOS:main Dec 10, 2024
3 checks passed
@ileana-pr
Copy link
Contributor Author

GM @jkbrooks thanks for approving! I just realized I put down node 20 instead of 23 for step 3. I also didn't write code to make the guide visible, I figures you guys would decide where it best fits in the docs. Cheers.

@jkbrooks
Copy link
Contributor

okay, reverted for now @ileana-pr #959

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