Skip to content

Update test packages#22

Merged
mtmk merged 1 commit intomainfrom
update-test-prj-pkgs
Mar 17, 2026
Merged

Update test packages#22
mtmk merged 1 commit intomainfrom
update-test-prj-pkgs

Conversation

@mtmk
Copy link
Copy Markdown
Member

@mtmk mtmk commented Mar 17, 2026

No description provided.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates test and benchmark package versions across the repository's test projects and adds consistent asset filtering to coverlet.collector in the NaCl test project.

Changes:

  • Bumps Microsoft.NET.Test.Sdk, coverlet.collector, and BenchmarkDotNet to newer versions across all relevant project files.
  • Adds <IncludeAssets> and <PrivateAssets> metadata to coverlet.collector in NATS.NKeys.NaCl.Tests.csproj to match the convention in NATS.NKeys.Tests.csproj.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
NATS.NKeys.Tests/NATS.NKeys.Tests.csproj Bumps Microsoft.NET.Test.Sdk to 18.3.0 and coverlet.collector to 8.0.1
NATS.NKeys.NaCl.Tests/NATS.NKeys.NaCl.Tests.csproj Bumps Microsoft.NET.Test.Sdk to 18.3.0 and coverlet.collector to 8.0.1 with asset filtering added
NATS.NKeys.Benchmarks/NATS.NKeys.Benchmarks.csproj Bumps BenchmarkDotNet to 0.15.8

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Copy Markdown
Member

@Jarema Jarema left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@mtmk mtmk merged commit 0b11e47 into main Mar 17, 2026
12 checks passed
@mtmk mtmk deleted the update-test-prj-pkgs branch March 17, 2026 11:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants