Skip to content

build(deps): lower shipped floors - #273

Merged
thomhurst merged 2 commits into
mainfrom
issue-190-dependency-floors
Aug 24, 2026
Merged

build(deps): lower shipped floors#273
thomhurst merged 2 commits into
mainfrom
issue-190-dependency-floors

Conversation

@thomhurst

Copy link
Copy Markdown
Owner

Closes #190.

Behavior

  • lowers shipped Microsoft.Extensions, RateLimiting, and TimeProvider dependency floors to .NET 8 versions
  • bounds Reservoir to [1.4.0, 2.0.0)
  • keeps test projects on .NET 10 packages through VersionOverride
  • prevents Renovate major/minor updates from raising shipped floors
  • verifies exact nuspec floors and a warning-clean net8 consumer using HTTP, DI/reloading, registry lookup, and rate limiting

Validation

  • dotnet build Kevlar.slnx -c Release
  • core tests: 845 passed
  • integration tests: 131 passed
  • analyzer tests: 78 passed
  • Testing tests: 51 passed
  • Chaos tests: 33 passed
  • rate-limiting tests: 24 passed
  • allocation tests: 3 passed
  • netstandard tests: 10 + 4 passed
  • Renovate config validated with the official CLI
  • full package verifier passed before final rebase, including the pinned net8 consumer and publish compatibility; final rerun reached only transient raw.githubusercontent.com propagation for the newly pushed SourceLink commit

@coderabbitai

coderabbitai Bot commented Aug 24, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: d688327f-5836-4df2-89e4-4e371be9c8d9


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@thomhurst
thomhurst force-pushed the issue-190-dependency-floors branch from 8bd0df7 to 0b6a36d Compare August 24, 2026 21:57

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 0b6a36dd7f

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread scripts/Verify-Packages.ps1 Outdated
@thomhurst
thomhurst force-pushed the issue-190-dependency-floors branch from 0b6a36d to c0e8257 Compare August 24, 2026 22:14

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: c0e8257c59

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread tests/Kevlar.Tests/Kevlar.Tests.csproj Outdated
@thomhurst
thomhurst force-pushed the issue-190-dependency-floors branch 2 times, most recently from 42d89f2 to ea62799 Compare August 24, 2026 22:59
Keep package consumers on the .NET 8 dependency line while test
projects retain .NET 10 coverage. Bound Reservoir below its next major.

Closes #190
@thomhurst
thomhurst force-pushed the issue-190-dependency-floors branch from ea62799 to e92fba2 Compare August 24, 2026 23:19
@thomhurst
thomhurst merged commit de46921 into main Aug 24, 2026
10 checks passed
@thomhurst
thomhurst deleted the issue-190-dependency-floors branch August 24, 2026 23:27
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.

build(deps): lower Microsoft.Extensions.* floor to 8.0.x and bound the Reservoir dependency

1 participant