-
Notifications
You must be signed in to change notification settings - Fork 4
Restore missing CLI pages (and other misc improvements) #134
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
…p /constructs/overview page
… page to /constructs/overview
…s/dynamic-monitor-creation
…or CLI env vars table
… Constructs sidebar order
…/docs into guolau/restore-cli-missing-pages
ragog
left a comment
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.
@guolau LGTM. One note: I am ok with removing the current Basic Workflow page as it is not what it should be. But I believe there is value in having a separate page giving a beginner a taste for what it can be used to, tying the different commands together into a workflow. LMK if you feel this is already covered, I might have missed it.
@ragog I would agree that this is valuable. It seems like we have a guide that covers this already, though it could use a bit of polishing: Getting Started with MaC guide. Added as a ticket: https://linear.app/checklyhq/issue/DOC-56/polish-the-getting-started-with-mac-guide-and-make-it-more-visible |
Affected Components
Summary of changes
Sections:
Sections:
Sections:
/cli/overview#using-ai-ides-and-copilots.Probably want to replace that tile with a more popular AI provider?Replaced it with Claude Code, since it's more popularOther notes
Ref: https://linear.app/checklyhq/issue/DOC-24/update-redirects-for-cli-urls-and-general-review-of-cli-pages