-
Notifications
You must be signed in to change notification settings - Fork 592
fix docs link #991
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
fix docs link #991
Conversation
@@ -13,7 +13,7 @@ Some of its key capabilities include: | |||
- Execute and fix tests, lint, and other commands | |||
- Search through git history, resolve merge conflicts, and create commits and PRs | |||
|
|||
**Learn more in the [official documentation](https://docs.anthropic.com/en/docs/agents/claude-code/introduction)**. | |||
**Learn more in the [official documentation](https://docs.anthropic.com/en/docs/claude-code/overview)**. | |||
|
|||
## Get started |
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.
L21 is also broken but it seems like those docs are just gone now, so not sure how you want to update it.
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.
i just entered "npm prefix" into the search, it shows up in two places, not sure which is best, but it can be found here:
https://docs.anthropic.com/en/docs/claude-code/settings#recommended-create-a-new-user-writable-npm-prefix
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.
i don't think my review has any privileges anyway, so i'll just leave these as comments, but LGTM
@@ -13,7 +13,7 @@ Some of its key capabilities include: | |||
- Execute and fix tests, lint, and other commands | |||
- Search through git history, resolve merge conflicts, and create commits and PRs | |||
|
|||
**Learn more in the [official documentation](https://docs.anthropic.com/en/docs/agents/claude-code/introduction)**. | |||
**Learn more in the [official documentation](https://docs.anthropic.com/en/docs/claude-code/overview)**. |
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.
nit:
**Learn more in the [official documentation](https://docs.anthropic.com/en/docs/claude-code/overview)**. | |
**Learn more in the [official documentation](https://docs.anthropic.com/en/docs/claude-code)**. |
This redirects to the /overview
but leaving it out is slightly more future-proof. otherwise LGTM
@@ -13,7 +13,7 @@ Some of its key capabilities include: | |||
- Execute and fix tests, lint, and other commands | |||
- Search through git history, resolve merge conflicts, and create commits and PRs | |||
|
|||
**Learn more in the [official documentation](https://docs.anthropic.com/en/docs/agents/claude-code/introduction)**. | |||
**Learn more in the [official documentation](https://docs.anthropic.com/en/docs/claude-code/overview)**. | |||
|
|||
## Get started |
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.
i just entered "npm prefix" into the search, it shows up in two places, not sure which is best, but it can be found here:
https://docs.anthropic.com/en/docs/claude-code/settings#recommended-create-a-new-user-writable-npm-prefix
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.
Thanks for the contribution!
No description provided.