Fixing readme - #2969
Conversation
|
🟡 Contributor Reputation Check: MEDIUM risk
Maintainers: please review this contributor before merging. |
github-actions
Bot
added
the
needs-review:MEDIUM
There was a problem hiding this comment.
Copilot review overview
🟡 Changes recommended
The skills README incorrectly advertises nonexistent obj/ build artifacts.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Review tier: Balanced
Findings: 1
New issues introduced by this change (1)
| Severity | Finding |
|---|---|
 |
docs/README.skills.md — The newly listed obj/ paths do not exist in the skill directory and are generated NuGet/MSBuild… |
What changed in this PR
Fixes generated catalog entries in the skills and agents documentation.
Changes:
- Restores missing catalog rows.
- Updates the Dynatrace MCP registry link.
- Accidentally lists nonexistent build artifacts for
winmd-api-search.
| File | Description |
|---|---|
docs/README.skills.md |
Restores the skill row but includes invalid generated assets. |
docs/README.agents.md |
Restores the Dynatrace row with its current registry link. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
There was a problem hiding this comment.
Copilot review overview
🟡 Changes recommended
The directory exclusion causes shipped skill assets to bypass documentation and validation.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Review tier: Balanced
Findings: 1
New issues introduced by this change (1)
| Severity | Finding |
|---|---|
|
eng/yaml-parser.mjs — This blanket name check also hides intentionally bundled files in any bin or obj directory (for… |
Issues resolved since last review (1)
| Severity | Finding |
|---|---|
|
docs/README.skills.md — The newly listed obj/ paths do not exist in the skill directory and are generated NuGet/MSBuild… View resolved comment |
No description provided.