Skip to content

Conversation

@MangelMaxime
Copy link
Member

@MangelMaxime MangelMaxime commented Nov 21, 2025

BuildAlyzer has a bug with .NET 10, there is a fix available since 1 month
but not sign of a release for now.

For this reason, we want to use a local version of BuildAlyzer with the fix included
and remove our own version later on.

See:
- Buildalyzer/Buildalyzer#318
- Buildalyzer/Buildalyzer#319

To build the custom version of Buildalyzer

  1. Clone https://github.com/Buildalyzer/Buildalyzer
  2. Comment all <PackageValidationBaselineVersion> lines (in the 3 projects). I was not able to make dotnet pack work with it
  3. Run dotnet build -c Release && dotnet pack -p:Version=8.0.0-fable-001

Copy link

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 fixes the BuildalyzerCracker to work with .NET 10 by upgrading to a custom local version of Buildalyzer (version 8.0.0-fable-001) that includes a fix from an unreleased upstream PR. The local packages are intended as a temporary workaround until an official release is available.

  • Updates Buildalyzer dependency from version 7.1.0 to 8.0.0-fable-001
  • Adds local NuGet package source pointing to LocalPackages/ directory
  • Includes 6 binary package files (3 .nupkg and 3 .snupkg files) for Buildalyzer components
  • Documents the fix in CHANGELOG.md

Reviewed changes

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

Show a summary per file
File Description
NuGet.Config Adds LocalPackages as a package source with documentation explaining the temporary workaround
src/Fable.Cli/Fable.Cli.fsproj Updates Buildalyzer package reference to version 8.0.0-fable-001
src/Fable.Cli/CHANGELOG.md Documents the fix in the Unreleased section
LocalPackages/*.nupkg Binary NuGet packages for Buildalyzer, Buildalyzer.Logger, and Buildalyzer.Workspaces
LocalPackages/*.snupkg Symbol packages for the three Buildalyzer components

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

MangelMaxime and others added 2 commits November 21, 2025 22:33
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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.

2 participants