Skip to content

[CI] Downgrade fsharp-analyzers to 0.35.0#4394

Merged
MangelMaxime merged 3 commits intomainfrom
dbrattli/fix-build
Mar 13, 2026
Merged

[CI] Downgrade fsharp-analyzers to 0.35.0#4394
MangelMaxime merged 3 commits intomainfrom
dbrattli/fix-build

Conversation

@dbrattli
Copy link
Collaborator

@dbrattli dbrattli commented Mar 12, 2026

Summary

  • Downgrade fsharp-analyzers from 0.36.0 to 0.35.0 to fix CI build failures

Test plan

  • Verify CI build passes with the downgraded version

🤖 Generated with Claude Code

@dbrattli
Copy link
Collaborator Author

dbrattli commented Mar 12, 2026

This fixes the build but I think this is may have been a result of fsharp-analyzers tool was bumped from 0.34.1 → 0.36.0 in #4389 and that it does not work well with FSharp.Analyzers.Build 0.5.0, so not sure the analyzers works anymore. Any ideas? @ncave @MangelMaxime

@MangelMaxime
Copy link
Member

If the problem is that FSharp.Analyzers.Build 0.5.0 broke because of the new version of fsharp-analyzers we should report it in https://github.com/ionide/FSharp.Analyzers.SDK.

I don't think this is normal that we need to do such work around in the CI pipeline

@dbrattli
Copy link
Collaborator Author

Should we then merge this until we can resolve the analyzer issue to get the builds passing again?

@MangelMaxime
Copy link
Member

After looking at it again, I am able to run the analyzers locally on my machine.

So it seems like the issue is related to .NET setup perhaps in the CI. I am trying to investigate in this direction

@ncave
Copy link
Collaborator

ncave commented Mar 13, 2026

@dbrattli

it does not work well with FSharp.Analyzers.Build 0.5.0

Can this be reproduced locally?

@dbrattli
Copy link
Collaborator Author

dbrattli commented Mar 13, 2026

Yes, I can reproduce it locally:

MSBuild version 18.0.2+b0f34d51f for .NET                                                                                                                        
    [15:31:02.259] [FSharp.Analyzers.Cli] error: Trying to load /Users/dbrattli/.nuget/packages/ionide.analyzers/0.14.11/analyzers/dotnet/fs/Ionide.Analyzers.dll  
  which was built using SDK version 0.35.0.0. Expect 0.36.0.0 instead. Assembly will be skipped.                                                                   
    [15:31:02.260] [FSharp.Analyzers.Cli] warn: Analyzer path /Users/dbrattli/Developer/fable/fix-build/src/Fable.Analyzers/bin/Release/net10.0 does not exist     
    [15:31:04.340] [FSharp.Analyzers.Cli] error: Because we failed to load some assemblies to obtain analyzers from them, exiting (failure count: 1) 

So the fix should be to downgrade fsharp-analyzers back to 0.35.0

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@dbrattli dbrattli force-pushed the dbrattli/fix-build branch from 734af5e to f7eaa03 Compare March 13, 2026 17:56
@dbrattli dbrattli changed the title [CI] Fix SARIF upload when reports directory is missing [CI] Downgrade fsharp-analyzers to 0.35.0 Mar 13, 2026
@MangelMaxime
Copy link
Member

With these changes now, it seems like we are able to run all the Analyzers against our project.

@MangelMaxime MangelMaxime merged commit 7650479 into main Mar 13, 2026
23 checks passed
@MangelMaxime MangelMaxime deleted the dbrattli/fix-build branch March 13, 2026 19:06
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.

3 participants