Daniel Cazzulino

Hey @dellis1972! Thanks for dropping by 🤟 . So perhaps the logic for the warning would be: you don't have a `GitDefaultBranch` property set (meaning next major will break you)....

You can't reference floating versions when packing. This is not supported in nuspec neither nuget packaging APIs, AFAIK. And it makes sense, since you would end up with non-deterministic packages...

Fair point. This is very low priority for me personally since I never use checked-in lock files, so I get my determinism from the actual fixed versions in the package...

I'm afraid I don't know what targets the MVC scaffolding are calling on why it would need (or even fail) the generated file. Could use help here.

If you're not using the tags for versioning, you can turn off tag processing entirely [GitIgnoreTagVersion](https://github.com/kzu/GitInfo/blob/master/src/GitInfo/build/GitInfo.targets#L44-L47). Or do you still use some tags for versioning and also for other purposes...

I know it's been ages, but is this still an issue with the latest version?

I've no idea why that target is being referenced, but it's part of the F# SDK: https://github.com/dotnet/fsharp/blob/main/src/FSharp.Build/Microsoft.FSharp.NetSdk.targets#L60 So, it doesn't look like it's something I can fix on my end....

I can't see how *removing* the double quotes is fixing the issue. Could you explain how/why it would work? (or even better, could you record a short gif that showcases...

Looks good! As soon as you sign the CLA, I'll merge this. Thanks for your contribution!

Would you mind rebasing on top of main to resolve conflicts?