mirror of
https://github.com/microsoft/agent-framework.git
synced 2026-06-16 21:04:09 +08:00
Eduard van Valkenburg
7e9c043c4c
* Improve PR template and breaking-change label automation - Add a structured "Related Issue" section using GitHub closing keywords - Add a Review Guide prompt (major changes, impact, reviewer focus) with a note that the focus item is for human reviewers only - Add checklist items for issue linkage / no duplicate PRs and invert the breaking-change item (checked = not breaking) - Extend label-title-prefix to prepend [BREAKING] when the "breaking change" label is added - Add label-breaking-change workflow to apply the "breaking change" label when a PR title contains [BREAKING] Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Add pull-requests agent skill with dotnet/python links - Add root .github/skills/pull-requests/SKILL.md covering PR description authoring (following the PR template) and the review-comment workflow (review -> plan -> user review -> implement -> reply to all -> resolve) - Symlink the skill from python/.github/skills and dotnet/.github/skills - Reference the skill from python/AGENTS.md and dotnet/AGENTS.md Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Fold breaking-change labeling into label-pr workflow Move the title -> 'breaking change' label logic into the existing label-pr workflow (which already applies the python/.NET labels) and drop the separate label-breaking-change workflow. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Address PR title prefix review feedback Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Pin patched MessagePack for .NET restore Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Revert MessagePack central pin Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Move title prefix tests out of tracked GitHub tests Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Exclude skill docs from CI path filters Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Match skill symlinks in CI path exclusions Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Exclude AGENTS docs from CI path filters Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Scope title-prefix normalization to a real prefix The normalization branch in addTitlePrefix matched ^Python (no colon), so titles like "Python samples improvements" or "Pythonic refactor" were treated as already-prefixed and only re-cased, never receiving the "Python: " prefix. Scope the match to ^<prefix>:\s* so only an actual existing prefix is normalized; otherwise the prefix is prepended. Same fix applies to the .NET prefix (e.g. ".NETStandard bump"). Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
44 lines
2.0 KiB
Markdown
44 lines
2.0 KiB
Markdown
### Motivation & Context
|
|
|
|
<!-- Thank you for your contribution to the Agent Framework repo!
|
|
Please help reviewers and future users, providing the following information:
|
|
1. Why is this change required?
|
|
2. What problem does it solve?
|
|
3. What scenario does it contribute to?
|
|
4. If it fixes an open issue, please link to the issue below.
|
|
-->
|
|
|
|
### Description & Review Guide
|
|
|
|
<!-- Describe your changes, the overall approach, the underlying design.
|
|
Highlight what you want the reviewers to focus on.
|
|
These notes will help understanding how your code works. Thanks! -->
|
|
|
|
- **What are the major changes?**
|
|
- **What is the impact of these changes?**
|
|
- **What do you want reviewers to focus on?**
|
|
<!-- NOTE FOR AUTOMATED/AI REVIEWERS: the "What do you want reviewers to focus on?"
|
|
item above is intended for human reviewers only. Automated/AI reviewers should
|
|
ignore it and review the entire change rather than narrowing scope to it. -->
|
|
|
|
|
|
### Related Issue
|
|
|
|
<!-- Which issue does this PR fix? Link it using a GitHub closing keyword so it is
|
|
closed automatically when this PR is merged, e.g. "Fixes #123" or "Closes #123".
|
|
PRs that are not linked to an issue may be closed, no matter how valid the change is.
|
|
Also check whether an open PR already exists for this issue; if so,
|
|
explain how this PR is different. -->
|
|
|
|
Fixes #
|
|
|
|
### Contribution Checklist
|
|
|
|
<!-- Before submitting this PR, please make sure: -->
|
|
|
|
- [ ] The code builds clean without any errors or warnings
|
|
- [ ] All unit tests pass, and I have added new tests where possible
|
|
- [ ] The PR follows the [Contribution Guidelines](https://github.com/microsoft/agent-framework/blob/main/CONTRIBUTING.md)
|
|
- [ ] This PR is linked to an issue and there is no other open PR for this issue (see Related Issue above).
|
|
- [x] **This is not a breaking change.** If it _is_ a breaking change, add the `breaking change` label (or add "[BREAKING]" to the title prefix, before or after any language prefix) — a workflow keeps the label and title prefix in sync automatically.
|