Skip to content

Add .NET 9.0 Support #269

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 8 commits into from
Apr 1, 2025
Merged

Add .NET 9.0 Support #269

merged 8 commits into from
Apr 1, 2025

Conversation

GeekInTheNorth
Copy link
Owner

Closes #264

@GeekInTheNorth GeekInTheNorth self-assigned this Apr 1, 2025
@GeekInTheNorth GeekInTheNorth requested a review from Copilot April 1, 2025 07:50
Copy link

@Copilot 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 pull request updates the project to support .NET 9.0 by renaming namespaces from "OptimizelyTwelveTest" to "OptiNetNine" and updating the CI workflow to build and test multiple .NET versions.

  • Updated all namespaces to reflect the new .NET 9.0 support
  • Enhanced the GitHub workflow to include build and test steps for .NET 6.0 and .NET 9.0
  • Adjusted branch filters for push and pull_request triggers in the workflow

Reviewed Changes

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

Show a summary per file
File Description
Sample/OptiNetNine/Features/GeneralContent/GeneralContentPageViewComponent.cs Updated namespace from OptimizelyTwelveTest to OptiNetNine
Sample/OptiNetNine/Features/GeneralContent/GeneralContentPageController.cs Updated namespace and using statements
Sample/OptiNetNine/Features/GeneralContent/GeneralContentPage.cs Updated namespace
Sample/OptiNetNine/Features/Common/UpperCaseNamingPolicy.cs Updated namespace
Sample/OptiNetNine/Features/Common/Pages/SitePageData.cs Updated namespace
Sample/OptiNetNine/Features/Common/Pages/ISitePageViewModel.cs Updated namespace
Sample/OptiNetNine/Features/Common/Pages/ISitePageData.cs Updated namespace
Sample/OptiNetNine/Features/Common/PageControllerBase.cs Updated namespace
Sample/OptiNetNine/Features/Common/Media/ImageContentViewComponent.cs Updated namespace
Sample/OptiNetNine/Features/Common/Media/ImageContent.cs Updated namespace
Sample/OptiNetNine/Features/Common/GroupNames.cs Updated namespace
Sample/OptiNetNine/Features/Blocks/RichText/RichTextViewComponent.cs Updated namespace
Sample/OptiNetNine/Features/Blocks/RichText/RichTextBlock.cs Updated namespace
Sample/OptiNetNine/Features/Blocks/Embed/EmbedBlockViewModel.cs Updated namespace
Sample/OptiNetNine/Features/Blocks/Embed/EmbedBlockViewComponent.cs Updated namespace
Sample/OptiNetNine/Features/Blocks/Embed/EmbedBlock.cs Updated namespace
Sample/OptiNetNine/Features/API/TestApiController.cs Updated namespace
.github/workflows/dotnet.yml Updated workflow configuration to use .NET 9.0 and added steps for .NET 6.0
Files not reviewed (1)
  • Sample/OptiNetNine.sln: Language not supported
Comments suppressed due to low confidence (2)

.github/workflows/dotnet.yml:5

  • The change in the push trigger branch filter restricts runs to feature branches only, which may unintentionally exclude commits pushed directly to main or develop. Please confirm if this change was intended.
branches: [ "feature/**" ]

.github/workflows/dotnet.yml:20

  • The inclusion of restore/build/test steps for .NET 6.0 in a PR focused on .NET 9.0 support might be unintended; please verify if maintaining support for .NET 6.0 is required.
run: dotnet restore Sample/OptiNetSix.sln

@GeekInTheNorth GeekInTheNorth merged commit c0e8eb7 into develop Apr 1, 2025
@GeekInTheNorth GeekInTheNorth deleted the feature/net_9_support branch April 1, 2025 07:58
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.

1 participant