docs: Migrate code review docs#1055
Conversation
|
Preview deployment for your docs. Learn more about Mintlify Previews.
💡 Tip: Enable Workflows to automatically generate PRs for you. |
|
Droid finished @TheFactoriousDROID's task —— View job LGTM — no high-confidence issues found in this docs-only change. |
| 3. Walk you through review configuration (depth, security, triggers) | ||
| 4. Create a PR/MR with the workflow files | ||
|
|
||
| You can also use `/install-github-app` for GitHub-only setup. For detailed setup instructions, see the [GitHub App installation guide](/integrations/github-app). |
There was a problem hiding this comment.
maybe we can keep this but replace the command with /install-code-review
There was a problem hiding this comment.
This page is the /install-code-review doc itself (already mentioned in line 24). The old github-app page redirects here.
|
|
||
| ## See also | ||
|
|
||
| - [GitHub App installation](/integrations/github-app) - Full setup guide for `/install-github-app` |
There was a problem hiding this comment.
This page is the Automated Code Review page already, we'd be redirecting to the current page. The github-app page was just an obsolete 3rd page (not local or automated, which we're keeping).
Old gh-app url will redirect to automated code review page
| droid exec "Review the changes in this PR for security issues and performance regressions" | ||
| ``` | ||
| See the [Code Review guide](/guides/droid-exec/code-review) for automation examples. | ||
| See the [Automated Code Review guide](/guides/droid-exec/code-review) for automation setup. |
There was a problem hiding this comment.
do we also want to move this page somewhere more discoverable/ top level
There was a problem hiding this comment.
Yes, this PR removes legacy github app page: https://docs.factory.ai/integrations/github-app
Moves Automated Code Rview up to top level capabilities https://docs.factory.ai/guides/droid-exec/code-review (https://factory-docs-code-review-docs-migration.mintlify.app/guides/droid-exec/code-review)
And just renames https://docs.factory.ai/cli/features/code-review (https://factory-docs-code-review-docs-migration.mintlify.app/cli/features/code-review)
There was a problem hiding this comment.
screenshot from the preview deploy
2 participants
Summary
/reviewdocs to Local Code Review and keep it as a top-level capability so local review and automated review are clearly separated./install-code-reviewis now the current setup path./install-code-reviewand the new Code Review naming.Risk / impact
Testing
npm run check/guides/droid-exec/code-review.