-
Notifications
You must be signed in to change notification settings - Fork 927
docs: reorganize and edit docs README #14706
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
Conversation
All contributors have signed the CLA ✍️ ✅ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, except a few comments. I don't need to review again.
I have read the CLA Document and I hereby sign the CLA |
In the coder/coder repo on your machine, run |
thanks @stirby - I had a series of errors when I ran full error text❯ make fmt ─╯
ERROR: You need GNU getopt to run the scripts in the Coder repo.
On darwin:
- brew install gnu-getopt
- Add "$(brew --prefix gnu-getopt)/bin" to your PATH
- Restart your terminal
ERROR: You need at least make 4.0 to run the scripts in the Coder repo.
On darwin:
- brew install make
- Add "$(brew --prefix make)/libexec/gnubin" to your PATH
- Restart your terminal
ERROR: Invalid dependencies, see above for more details.
==> fmt/ts
ERR_PNPM_NO_SCRIPT Missing script: check:fix
Command "check:fix" not found.
make: *** [fmt/ts] Error 1 once I figured out it's just running prettier, I used npx instead and that seemed to do the trick: |
@EdwardAngert Edward You can also create a workspace from the dogfood template and run
|
0f6085a
to
66d75f8
Compare
I think it would be easy if you could redo this PR @EdwardAngert. cc: @stirby |
27d23c3
to
3469242
Compare
Edit the docs readme for consistency and better flow