Skip to content
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

Latest F# compiler warns on autogenerated Version.fs #1173

Closed
Smaug123 opened this issue Feb 14, 2025 · 3 comments
Closed

Latest F# compiler warns on autogenerated Version.fs #1173

Smaug123 opened this issue Feb 14, 2025 · 3 comments
Milestone

Comments

@Smaug123
Copy link

Smaug123 commented Feb 14, 2025

dotnet/fsharp#17871 made it a compile warning to specify an invalid #nowarn suppression in an F# file. The default Version.fs generated file contains a nowarn, CA2243 (sorry, I'm on my phone, will provide more details and possibly a fix when I get to a real computer).

I believe the correct fix is simply not to nowarn in the F# generated file.

@Numpsy
Copy link
Contributor

Numpsy commented Feb 14, 2025

Same here fwiw:

PackagingTools\artifacts\obj\PackageViewer\release_win-x64\PackageViewer.Version.fs(11,9): warning FS0203: Invalid warning number 'CA2243'

CA2243 looks like a Roslyn analyzer, so I don't think it would run on F# code anyway

@Smaug123
Copy link
Author

Thanks for the fix! @AArnott What's the release process for this? I can of course work around the bug, but last night Dependabot's automatic PRs became blocked in most of my repos when they attempted to upgrade the .NET SDK. Would it be easy to cut a release of Nerdbank.GitVersioning?

@AArnott
Copy link
Collaborator

AArnott commented Feb 24, 2025

@Smaug123 You can always consume our CI builds as described here.

If you need the package on nuget.org, I could ship a 3.8 pre-release to nuget.org. Or if you need a stable versioned release we could backport the fix to 3.7.

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

No branches or pull requests

3 participants