Skip to content

build: Target .NET 8 and .NET 9 - #8

Merged
hangy merged 4 commits into
devfrom
target-net8.0
Sep 6, 2025
Merged

build: Target .NET 8 and .NET 9#8
hangy merged 4 commits into
devfrom
target-net8.0

Conversation

@hangy

@hangy hangy commented Sep 4, 2025

Copy link
Copy Markdown
Contributor

No description provided.

@hangy
hangy requested a review from Copilot September 4, 2025 06:52
@hangy hangy self-assigned this Sep 4, 2025

This comment was marked as outdated.

@codecov

codecov Bot commented Sep 4, 2025

Copy link
Copy Markdown

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

@hangy
hangy requested a review from Copilot September 4, 2025 07:04

Copilot AI 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.

Pull Request Overview

This PR updates the project to multi-target both .NET 8 and .NET 9, expanding from the previous .NET 9-only targeting. The change enables broader compatibility while maintaining support for the latest .NET version.

  • Changes project targeting from single framework (net9.0) to multi-framework (net8.0;net9.0)
  • Updates package lock files to include dependency resolution for both target frameworks
  • Maintains existing functionality while expanding platform support

Reviewed Changes

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

Show a summary per file
File Description
Directory.Build.props Updated TargetFramework to TargetFrameworks with both net8.0 and net9.0
tests/AlbusKavaliro.TraceableLdapClient.Tests/packages.lock.json Added net8.0 dependency resolution alongside existing net9.0
src/AlbusKavaliro.TraceableLdapClient/packages.lock.json Added net8.0 dependency resolution alongside existing net9.0
example/WorkerService/packages.lock.json Added net8.0 dependency resolution alongside existing net9.0
example/ServiceDefaults/packages.lock.json Added net8.0 dependency resolution alongside existing net9.0
example/AppHost/packages.lock.json Added net8.0 dependency resolution alongside existing net9.0

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Autobuild doesn't install .NET 8 and .NET 9, which is required for this kind of multi-targeting.
@hangy
hangy merged commit 4c493b6 into dev Sep 6, 2025
9 checks passed
@hangy
hangy deleted the target-net8.0 branch September 6, 2025 07:13
@github-actions

github-actions Bot commented Sep 6, 2025

Copy link
Copy Markdown

🎉 This PR is included in version 1.0.0-beta.4 🎉

The release is available on:

Your semantic-release bot 📦🚀

@github-actions

Copy link
Copy Markdown

🎉 This PR is included in version 1.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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.

2 participants