Restore techdebt review#11965
Conversation
There was a problem hiding this comment.
More details
This PR updates documentation and skill metadata only — no runtime code changes. It restores the /techdebt review guidance to the workflow by improving skill descriptions and reorganizing review guidelines in AGENTS.md. All changes are non-runtime and carry no behavioral risk.
🤖 Datadog Autotest · Commit 3f8cde5 · What is Autotest? · Any feedback? Reach out in #autotest
🟢 Java Benchmark SLOs — All performance SLOs passed
PR vs. master results
Commit: Load and DaCapo benchmarks can be triggered manually in the GitLab pipeline. Results will appear in the Benchmarking Platform UI after completion. |
dougqh
left a comment
There was a problem hiding this comment.
I've had rather mixed results with /tech-debt on some of my performance toolkit PRs, but none-the-less, I think having this enabled by default is probably for the better.
Did it went against the performance improvements? Or is it more about the general feedback it provided was low quality? |
|
Closing in favor of #12003 |
What Does This Do
This PR restores the
/techdebtreview to avoid pushing bloated code.Motivation
The performance review removed the design review.
Additional Notes
Update the skill description to the new standards.
Contributor Checklist
type:and (comp:orinst:) labels in addition to any other useful labelsclose,fix, or any linking keywords when referencing an issueUse
solvesinstead, and assign the PR milestone to the issueJira ticket: APMLP-1609