docs: add Projects section to Monitoring as Code concept page - #508
Merged
Merged
Conversation
Contributor
|
Preview deployment for your docs. Learn more about Mintlify Previews.
|
Contributor
There was a problem hiding this comment.
🟡 Changes recommended
Scope the Managed by claim and restore the missing punctuation.
Get a fresh assessment by requesting another Copilot review.
Pull request overview
Adds a Projects subsection to the Monitoring as Code concept page, covering project configuration, independent deployments, and UI ownership metadata.
Changes:
- Documents projects and multi-project accounts.
- Explains the Managed by UI section.
- Adds supporting project screenshots and updates the introduction.
File summaries
| File | Summary | Findings |
|---|---|---|
concepts/monitoring-as-code.mdx |
Adds Projects documentation and screenshot references. | Moderate (2 votes): Scope the Managed by claim to supported resource types. Nit (3 votes): Restore punctuation in the introduction. |
Review details
- Files reviewed: 1/3 changed files
- Comments generated: 2
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Correct punctuation in the definition of Monitoring as Code. Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Affected Components
Notes for the Reviewer
Adds a
### Projectssubsection under "Core Principles of Monitoring as Code" onconcepts/monitoring-as-code.mdx. It covers:checkly.config.ts/.jsvia the Project constructcheckly deployTwo screenshots added under
images/: the Projects list page and the Managed by row on a check.Also included in this diff: a pre-existing local change to the intro sentence ("code artifactversion controlled" became "code artifact version controlled"). The original was likely a lost em-dash, so the sentence may still want a colon or comma there. Happy to adjust in this PR.
Open questions I did not write into the page: whether CLI-managed resources are editable in the UI (the screenshot shows an Edit button), and whether every resource type gets the Managed by section or only checks and groups.
Screenshots
See the two new images in
images/projects-overview.pngandimages/projects-managed-by.png.