Skip to content

Conversation

@d-bytebase
Copy link
Contributor

Summary

  • Removed Server Startup Options documentation page to provide fewer configuration possibilities
  • Cleaned up references and simplified Docker deployment guide

Changes

  • Deleted /mintlify/reference/command-line.mdx
  • Removed navigation entry from docs.json
  • Removed "Advanced Options" section from Docker deployment guide
  • Simplified Colima support section by removing unnecessary Docker run example

Test plan

  • Verify documentation builds without errors
  • Check all internal links still work
  • Confirm Docker deployment guide is still clear without the removed sections

🤖 Generated with Claude Code

Remove Server Startup Options page and references to reduce configuration complexity by:
- Deleting the command-line.mdx reference page
- Removing navigation entry from docs.json
- Removing Advanced Options section from Docker deployment guide
- Simplifying Colima support section to only show essential setup

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
Copilot AI review requested due to automatic review settings August 12, 2025 13:37
@vercel
Copy link

vercel bot commented Aug 12, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Project Deployment Preview Comments Updated (UTC)
bytebase-com Ready Preview Comment Aug 12, 2025 1:45pm

@github-actions
Copy link

github-actions bot commented Aug 12, 2025

📝 Broken Links Check Report

Note: Entries under /api-reference/ are likely false positives and can be ignored.

The following is the output from the broken links check:

found 16 broken links in 9 files

integrations/api/audit-log.mdx
 ⎿  /api-reference/auditlogservice/post-v1auditlogs:search

integrations/api/authentication.mdx
 ⎿  /api-reference/authservice/post-v1authlogin

integrations/api/data-classification.mdx
 ⎿  /api-reference/settingservice/patch-v1settings
 ⎿  /api-reference/databaseservice/patch-v1instances-databases

integrations/api/issue.mdx
 ⎿  /api-reference/issueservice/post-v1projects-issues

integrations/api/permission.mdx
 ⎿  /api-reference/workspaceservice/get-v1workspaces-:getIamPolicy
 ⎿  /api-reference/projectservice/get-v1projects-:getIamPolicy
 ⎿  /api-reference/roleservice/get-v1roles
 ⎿  /api-reference/userservice/get-v1users
 ⎿  /api-reference/groupservice/get-v1groups

integrations/api/plan.mdx
 ⎿  /api-reference/planservice/post-v1projects-plans
 ⎿  /api-reference/sheetservice/post-v1projects-sheets

integrations/api/release.mdx
 ⎿  /api-reference/releaseservice/post-v1projects-releases

integrations/api/rollout.mdx
 ⎿  /api-reference/rolloutservice/post-v1projects-rollouts

integrations/api/sql-review.mdx
 ⎿  /api-reference/releaseservice/post-v1projects-releases:check
 ⎿  /api-reference/sqlservice/post-v1sqlcheck

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR removes server startup options documentation to simplify configuration choices and streamline the documentation. The changes eliminate the command-line reference page and clean up related references in the deployment guide.

  • Deleted the server startup options documentation page
  • Removed navigation references and simplified Docker deployment instructions
  • Cleaned up the Colima support section by removing unnecessary examples

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
mintlify/reference/command-line.mdx Complete removal of server startup options documentation
mintlify/get-started/self-host/deploy-with-docker.mdx Removed advanced options section and simplified Colima example
mintlify/docs.json Removed navigation entry for the deleted command-line reference

Only show the mirror image URL instead of the full Docker command

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
Remove reference to deleted command-line documentation page

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
Copy link
Member

@tianzhou tianzhou left a comment

Choose a reason for hiding this comment

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

LGTM

@d-bytebase d-bytebase merged commit 1c1bb0b into main Aug 12, 2025
5 checks passed
@d-bytebase d-bytebase deleted the remove-server-startup-options branch August 12, 2025 14:11
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