Skip to content

Add Copilot code review skill - #3247

Merged
martincostello merged 3 commits into
mainfrom
add-code-review-skill
Sep 29, 2026
Merged

martincostello merged 3 commits into
mainfrom
add-code-review-skill

Conversation

@martincostello

Copy link
Copy Markdown
Member
  • Add GitHub Copilot code review skills.
  • Fix benchmarks failing to build.
  • Extend the benchmark script with more options.

Add GitHub Copilot code review skills.
- Fix benchmarks failing to build.
- Extend the benchmark script with more options.
@codecov

codecov Bot commented Sep 29, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.02%. Comparing base (dc933fd) to head (b7410dc).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3247   +/-   ##
=======================================
  Coverage   98.02%   98.02%           
=======================================
  Files         321      321           
  Lines        7099     7099           
  Branches     1089     1089           
=======================================
  Hits         6959     6959           
  Misses         98       98           
  Partials       42       42           
Flag Coverage Δ
linux 97.97% <ø> (ø)
macos 98.01% <ø> (ø)
windows 98.02% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

Comment thread AGENTS.md
Reference the PR template.
@martincostello
martincostello marked this pull request as ready for review September 29, 2026 12:33
@martincostello
martincostello enabled auto-merge (squash) September 29, 2026 12:33
@martincostello
martincostello merged commit bb986e4 into main Sep 29, 2026
28 checks passed
@martincostello
martincostello deleted the add-code-review-skill branch September 29, 2026 12:34
SonnyRR pushed a commit to SonnyRR/stacked-deck-templates that referenced this pull request Oct 26, 2026
Updated [Polly](https://github.com/App-vNext/Polly) from 8.8.0 to 8.8.1.

<details>
<summary>Release notes</summary>

_Sourced from [Polly's
releases](https://github.com/App-vNext/Polly/releases)._

## 8.8.1

## Highlights

* Prevent the circuit breaker from capturing the ambient
ExecutionContext by @​Nacl-HCl in
App-vNext/Polly#3284
* Improve pipeline execution performance by @​martincostello in
App-vNext/Polly#3287
* Fix cleanup of discarded registry pipelines by @​harshvarudkar in
App-vNext/Polly#3288

## What's Changed

* Add Copilot code review skill by @​martincostello in
App-vNext/Polly#3247
* Fix benchmarks by @​martincostello in
App-vNext/Polly#3248
* Update PR template by @​martincostello in
App-vNext/Polly#3249
* Update README with OSMF details for Polly by @​joelhulen in
App-vNext/Polly#3263
* Fix lint warnings by @​martincostello in
App-vNext/Polly#3264
* Update FUNDING by @​martincostello in
App-vNext/Polly#3258
* Remove obsolete packages references by @​martincostello in
App-vNext/Polly#3267
* Validate packages in parallel by @​martincostello in
App-vNext/Polly#3283
* .NET 11 preparation by @​martincostello in
App-vNext/Polly#3285

## New Contributors

* @​Nacl-HCl made their first contribution in
App-vNext/Polly#3284
* @​harshvarudkar made their first contribution in
App-vNext/Polly#3288

**Full Changelog**:
App-vNext/Polly@8.8.0...8.8.1

Commits viewable in [compare
view](App-vNext/Polly@8.8.0...8.8.1).
</details>

[![Dependabot compatibility
score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=Polly&package-manager=nuget&previous-version=8.8.0&new-version=8.8.1)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)

Dependabot will resolve any conflicts with this PR as long as you don't
alter it yourself. You can also trigger a rebase manually by commenting
`@dependabot rebase`.

[//]: # (dependabot-automerge-start)
[//]: # (dependabot-automerge-end)

---

<details>
<summary>Dependabot commands and options</summary>
<br />

You can trigger Dependabot actions by commenting on this PR:
- `@dependabot rebase` will rebase this PR
- `@dependabot recreate` will recreate this PR, overwriting any edits
that have been made to it
- `@dependabot show <dependency name> ignore conditions` will show all
of the ignore conditions of the specified dependency
- `@dependabot ignore this major version` will close this PR and stop
Dependabot creating any more for this major version (unless you reopen
the PR or upgrade to it yourself)
- `@dependabot ignore this minor version` will close this PR and stop
Dependabot creating any more for this minor version (unless you reopen
the PR or upgrade to it yourself)
- `@dependabot ignore this dependency` will close this PR and stop
Dependabot creating any more for this dependency (unless you reopen the
PR or upgrade to it yourself)


</details>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant