docs: use emoji markers in target summary matrix - #2701
Open
Bruno Borges (brunoborges) wants to merge 1 commit into
Open
docs: use emoji markers in target summary matrix#2701Bruno Borges (brunoborges) wants to merge 1 commit into
Bruno Borges (brunoborges) wants to merge 1 commit into
Conversation
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Bruno Borges (brunoborges)
requested review from
Daniel Meppiel (danielmeppiel) and
Sergio Sisternes (sergio-sisternes-epam)
as code owners
August 26, 2026 21:24
Copilot started reviewing on behalf of
Bruno Borges (brunoborges)
August 26, 2026 21:24
View session
Contributor
There was a problem hiding this comment.
Pull request overview
Updates the documentation "targets summary matrix" to use more visually distinct markers for supported vs unsupported capabilities across APM targets.
Changes:
- Replaces
[x]/[ ]table markers with Unicode checkmark / cross symbols while keeping the same support states. - Preserves the existing IntelliJ footnote markers (
(*)) in the table.
Show a summary per file
| File | Description |
|---|---|
| docs/src/content/docs/reference/targets-matrix.md | Swaps the support/unsupported indicators in the summary matrix to improve scanability while keeping the same target capability mapping. |
Review details
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
- Files reviewed: 1/1 changed files
- Comments generated: 0
- Review effort level: Lite
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.
Description
The targets summary matrix used text-like checkbox markers that were slower to distinguish at a glance. This change replaces supported and unsupported markers with ✅ and ❌ while preserving every support state and the IntelliJ footnotes.
Trade-off: This intentionally introduces Unicode into this documentation table to provide the requested visual scanability; no CLI output or source behavior changes.
N/A - no linked issue.
Type of change
Testing
Validation: the full CI lint mirror passed with Ruff, Ruff formatting, pylint duplicate-code detection, and the auth-signal boundary check.
Spec conformance (OpenAPM v0.1)
If this PR changes behaviour that an OpenAPM v0.1
req-XXXcovers,confirm the three-step ritual (see CONTRIBUTING.md "Adding or
changing a normative requirement"):
docs/src/content/docs/specs/openapm-v0.1.mdupdated(new/changed
<a id="req-XXX"></a>anchor + prose + Appendix Crow).
docs/src/content/docs/specs/manifests/openapm-v0.1.requirements.ymlupdated.
@pytest.mark.req("req-XXX")test undertests/spec_conformance/added or extended.CONFORMANCE.{md,json}regenerated viauv run --extra dev python -m tests.spec_conformance.gen_statementand committed.
Co-authored-by: Copilot 223556219+Copilot@users.noreply.github.com