-
Notifications
You must be signed in to change notification settings - Fork 560
update mint.json #1955
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
update mint.json #1955
Conversation
Important Review skippedBot user detected. To trigger a single review, invoke the You can disable this status message by setting the Note Free review on us!CodeRabbit is offering free reviews until Wed May 28 2025 to showcase some of the refinements we've made. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
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.
PR Summary
Updates to docs/mint.json
navigation structure contain syntax and organizational issues that need to be addressed.
- Duplicate entry for
deploy-helm
in the 'Self-host Digger' section needs to be removed - Missing comma after
terraform.lock
in Reference section causing JSON syntax error self-host-on-azure
should be moved from Reference section to 'Self-host Digger' section for better organization
💡 (2/5) Greptile learns from your feedback when you react with 👍/👎!
1 file(s) reviewed, 3 comment(s)
Edit PR Review Bot Settings | Greptile
"ce/self-host/deploy-helm", | ||
"ce/self-host/deploy-helm" |
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.
logic: Duplicate entry 'deploy-helm' in the Self-host Digger section
"ce/self-host/deploy-helm", | |
"ce/self-host/deploy-helm" | |
"ce/self-host/deploy-helm" |
"ce/reference/terraform.lock" | ||
"ce/self-host/self-host-on-azure" |
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.
syntax: Missing comma after 'terraform.lock'
"ce/reference/terraform.lock" | |
"ce/self-host/self-host-on-azure" | |
"ce/reference/terraform.lock", | |
"ce/self-host/self-host-on-azure" |
@@ -120,6 +121,7 @@ | |||
"ce/reference/action-inputs", | |||
"ce/reference/api", | |||
"ce/reference/terraform.lock" | |||
"ce/self-host/self-host-on-azure" |
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.
style: 'self-host-on-azure' documentation should be moved to the Self-host Digger section for better organization
No description provided.