Skip to content

Rebase v2 on main #1727

New issue

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

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

Already on GitHub? # to your account

Open
wants to merge 8 commits into
base: v2
Choose a base branch
from
Open

Rebase v2 on main #1727

wants to merge 8 commits into from

Conversation

bUnitBot
Copy link
Contributor

This PR is created automatically by the bUnit bot.

When completing this PR, it's important to use Rebase and merge to keep the commit history clean.

linkdotnet and others added 8 commits June 14, 2025 16:17
* chore: Upgrade dependencies

* revert upgrade of AngleSharp as it leads to runtime issues

* Only target net80 and above for ci/cd

* Remove generators from the build
Release of new minor version v1.40
@linkdotnet linkdotnet requested a review from Copilot June 14, 2025 19:00
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 PR is an automated rebase by the bUnit bot to align the commit history with main and update various project files. Key changes include:

  • Bumping the project version in version.json and updating the changelog.
  • Updating target frameworks and removing project references and packing steps for bunit.generators.
  • Updating package versions and CI/CD workflows to support only net8.0 and net9.0.

Reviewed Changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
version.json Updated project version from "1.40-preview" to "1.41-preview", which appears misaligned with the changelog.
tests/bunit.generators.tests/bunit.generators.tests.csproj Changed target framework from a single net8.0 to multiple frameworks and removed a ProjectReference to bunit.generators.
bunit.sln Removed project entries related to bunit.generators, reflecting its removal from the solution.
Directory.Packages.props Upgraded various NuGet packages and adjusted System.Text.Json versions across TFMs.
CHANGELOG.md Added a new changelog entry for version [1.40.0] which appears inconsistent with version.json.
.github/workflows/release.yml, docs-deploy.yml, ci.yml Removed support for older .NET versions and updated packing/test steps to target net8.0 and net9.0.
Comments suppressed due to low confidence (5)

tests/bunit.generators.tests/bunit.generators.tests.csproj:28

  • Removal of the bunit.generators project reference may affect test coverage for generator logic. Confirm that this removal is intentional and that related tests have been updated accordingly.
<ProjectReference Include="..\..\src\bunit.generators\bunit.generators.csproj" OutputItemType="Analyzer" ReferenceOutputAssembly="false" />

bunit.sln:64

  • The removal of bunit.generators.tests and bunit.generators from the solution should be reviewed to ensure all dependent configurations and references have been updated.
Project("{9A19103F-16F7-4668-BE54-9A1E7A4F7556}") = "bunit.generators.tests", "tests\bunit.generators.tests\bunit.generators.tests.csproj", "{09046981-D9EC-4295-8502-721AC54E1F12}"

.github/workflows/release.yml:100

  • The removal of the dotnet pack command for bunit.generators indicates that this project is no longer being published. Ensure this change is consistent with the overall solution updates.
dotnet pack src/bunit.generators/ -c Release --property:PackageOutputPath=${GITHUB_WORKSPACE}/packages -p:ContinuousIntegrationBuild=true -p:publicrelease=true

.github/workflows/ci.yml:77

  • Removing the bunit.generators pack step from the CI workflow should be confirmed to match the intended project structure and packaging strategy.
dotnet pack src/bunit.generators/ -c release --output ${{ env.NUGET_DIRECTORY }} -p:ContinuousIntegrationBuild=true -p:publicrelease=true

Directory.Packages.props:99

  • System.Text.Json is set to version 9.0.5 for net9.0 while other frameworks remain on 8.0.5. Verify that this discrepancy is intentional and supported by the project requirements.
<PackageVersion Include="System.Text.Json" Version="9.0.5"/>

@@ -1,6 +1,6 @@
{
"$schema": "https://raw.githubusercontent.com/dotnet/Nerdbank.GitVersioning/main/src/NerdBank.GitVersioning/version.schema.json",
"version": "1.40-preview",
"version": "1.41-preview",
Copy link
Preview

Copilot AI Jun 14, 2025

Choose a reason for hiding this comment

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

The version in version.json ("1.41-preview") does not match the changelog entry ([1.40.0]). Consider aligning these to avoid confusion.

Suggested change
"version": "1.41-preview",
"version": "1.40.0",

Copilot uses AI. Check for mistakes.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants