-
Notifications
You must be signed in to change notification settings - Fork 3.8k
docs: reorder mission-control sections in docs.json #8821
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
Move Integrations section after workflows for better logical flow. Now the order is: tasks -> workflows -> integrations -> metrics -> sharing. Co-authored-by: bekah-hawrot-weigel <bekah@continue.dev> Generated with [Continue](https://continue.dev) Co-Authored-By: Continue <noreply@continue.dev>
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.
No issues found across 1 file
CI Test Failures - Not Related to ChangesThe failing macOS tests are infrastructure issues unrelated to the docs.json changes: Root Causes:
Evidence:
Retriggering CI by pushing empty commit to retry the flaky macOS tests. |
macOS test failures are unrelated to docs.json changes: - Sharp download 503 errors (transient network issue) - Flaky TUIChat.editMessage test Co-authored-by: bekah-hawrot-weigel <bekah@continue.dev> Generated with [Continue](https://continue.dev) Co-Authored-By: Continue <noreply@continue.dev>
Request for Maintainer ReviewThe macOS CLI tests continue to fail due to infrastructure issues unrelated to this docs-only PR: Issue Details:
Successful Checks (50+):✅ Mintlify Deployment - Docs build and deploy successful Failed Checks:❌ macOS Node 18 - Sharp download 503 error Comparison:Recent docs PR #8797 passed all checks. The difference is timing - GitHub/npm registry had intermittent availability issues. Recommended Action:Since this is a docs-only change with Mintlify validation passing, could a maintainer either:
cc @maintainers |
Root Cause Analysis & SolutionCurrent SituationThis PR modifies only ProblemThe on:
pull_request:
branches: [main]This causes unnecessary test runs for docs-only changes and blocks PRs when unrelated infrastructure fails. Proposed SolutionAdd path filtering to on:
pull_request:
branches: [main]
paths-ignore:
- 'docs/**'
- '*.md'
- '.github/workflows/preview.yaml'Benefits
Immediate Action NeededFor this PR specifically:
Recommendation: Merge this PR and create a follow-up to add path filtering to the workflow. Would a maintainer be able to approve/merge despite the flaky macOS tests? |
|
🎉 This PR is included in version 1.6.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
|
🎉 This PR is included in version 1.36.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Summary
Moved the Integrations section to appear directly after Workflows in the Mission Control navigation menu for better logical flow and user experience.
Changes
docs/docs.jsonThis makes the navigation more intuitive as users typically configure workflows before setting up integrations.
This agent session was co-authored by bekah-hawrot-weigel and Continue.
Summary by cubic
Reordered Mission Control navigation so Integrations appears directly after Workflows for a clearer setup flow and easier discovery.
Written for commit 803294f. Summary will update automatically on new commits.