Skip to content

Have tests run under any TFM#293

Merged
nojaf merged 3 commits intomainfrom
runtime-tfm-detection
Mar 19, 2026
Merged

Have tests run under any TFM#293
nojaf merged 3 commits intomainfrom
runtime-tfm-detection

Conversation

@Smaug123
Copy link
Contributor

I use net9 locally.

open FSharp.Analyzers.SDK

let mutable projectOptions: FSharpProjectOptions = FSharpProjectOptions.zero
let mutable runtimeTfm: string = Unchecked.defaultof<_>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Has to be initialised inside SetUp, because of lazy initialisation of static members.

let mutable runtimeTfm: string = Unchecked.defaultof<_>

[<SetUp>]
[<OneTimeSetUp>]
Copy link
Contributor Author

Choose a reason for hiding this comment

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

No need to redo this before every test!

Copy link
Contributor

@nojaf nojaf left a comment

Choose a reason for hiding this comment

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

Thanks Patrick!

@nojaf nojaf merged commit 071e631 into main Mar 19, 2026
2 checks passed
@Smaug123 Smaug123 deleted the runtime-tfm-detection branch March 19, 2026 11:13
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